Skip to content

Commit bc4a6b3

Browse files
wpanpaigeale
authored andcommitted
Simple cleanup, NFC.
Change-Id: Ib44d06e5573573537652d0b96cd9c7b9d96e925d
1 parent 1146fe6 commit bc4a6b3

File tree

1 file changed

+8
-14
lines changed

1 file changed

+8
-14
lines changed

IGC/Compiler/Optimizer/OpenCLPasses/GenericAddressResolution/GenericAddressDynamicResolution.cpp

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,6 @@ char GenericAddressAnalysis::ID = 0;
7979

8080
bool GenericAddressAnalysis::runOnFunction(Function& F)
8181
{
82-
bool HasGAS = false;
83-
8482
for (auto& BB : F.getBasicBlockList()) {
8583
for (auto& Inst : BB.getInstList()) {
8684
Type* Ty = Inst.getType();
@@ -94,22 +92,18 @@ bool GenericAddressAnalysis::runOnFunction(Function& F)
9492
Ty = GEP->getPointerOperandType();
9593
auto PT = dyn_cast<PointerType>(Ty);
9694
if (PT && PT->getAddressSpace() == ADDRESS_SPACE_GENERIC) {
97-
HasGAS = true;
98-
break;
95+
auto implicitArgs = ImplicitArgs(F, getAnalysis<MetaDataUtilsWrapper>().getMetaDataUtils());
96+
SmallVector<ImplicitArg::ArgType, 3> args;
97+
args.push_back(ImplicitArg::LOCAL_MEMORY_STATELESS_WINDOW_START_ADDRESS);
98+
args.push_back(ImplicitArg::LOCAL_MEMORY_STATELESS_WINDOW_SIZE);
99+
args.push_back(ImplicitArg::PRIVATE_MEMORY_STATELESS_SIZE);
100+
ImplicitArgs::addImplicitArgs(F, args, getAnalysis<MetaDataUtilsWrapper>().getMetaDataUtils());
101+
return true;
99102
}
100103
}
101104
}
102105

103-
if (HasGAS) {
104-
auto implicitArgs = ImplicitArgs(F, getAnalysis<MetaDataUtilsWrapper>().getMetaDataUtils());
105-
SmallVector<ImplicitArg::ArgType, 2> args;
106-
args.push_back(ImplicitArg::LOCAL_MEMORY_STATELESS_WINDOW_START_ADDRESS);
107-
args.push_back(ImplicitArg::LOCAL_MEMORY_STATELESS_WINDOW_SIZE);
108-
args.push_back(ImplicitArg::PRIVATE_MEMORY_STATELESS_SIZE);
109-
ImplicitArgs::addImplicitArgs(F, args, getAnalysis<MetaDataUtilsWrapper>().getMetaDataUtils());
110-
}
111-
112-
return HasGAS;
106+
return false;
113107
}
114108

115109
namespace {

0 commit comments

Comments
 (0)