[PATCH] D31103: [AMDGPU] Add address space based alias analysis pass

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 17 15:04:10 PDT 2017


rampitec added inline comments.


================
Comment at: lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp:44
+ /* Global   */ {NoAlias , MayAlias, NoAlias , NoAlias , MayAlias},
+ /* Constant */ {NoAlias , NoAlias , MayAlias, NoAlias , NoAlias },
+ /* Group    */ {NoAlias , NoAlias , NoAlias , MayAlias, MayAlias},
----------------
arsenm wrote:
> flat and constant may alias, this isn't the OpenCL rules
I think in the future as an extension module metadata can be checked to see if source was OpenCL and return NoAlias for flat and constant.


Repository:
  rL LLVM

https://reviews.llvm.org/D31103





More information about the llvm-commits mailing list