This is direct port of HSAILAliasAnalysis pass, just cleaned for style and renamed.
Details
Diff Detail
- Repository
- rL LLVM
Event Timeline
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
63 | the check can be done using xor T1->isVectorTy() ^ T2->isVectorTy() |
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
1 | Missing comment | |
13 | doesn't match pass name | |
15–17 | Should move this to the TargetMachine when adding it | |
44 | flat and constant may alias, this isn't the OpenCL rules | |
58–59 | Use cast<> if not checking it | |
71 | formatting | |
78 | Missin space | |
lib/Target/AMDGPU/AMDGPUAliasAnalysis.h | ||
1–2 | Missing comment |
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
75–81 | We should probably be marking all constant loads in the frontend with invariant metadata. | |
83–101 | I think BasicAA will handle all of this, we should only need to handle the address spaces | |
test/CodeGen/AMDGPU/vectorize-global-local.ll | ||
15–22 | Should run instnamer on this |
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
83–101 | It was not handled really, that's why it got here... |
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
83–101 | I think this is only correct to do if F is a kernel |
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
83–101 | You probably right, that is safer. I will change it. BTW, I have checked this method in BasicAA, it does not check argument attributes. |
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
44 | 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. |
Missing comment