Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM. Cleanup comments can be done as a separate patch at a later time.
lib/Target/AMDGPU/AMDGPUAliasAnalysis.cpp | ||
---|---|---|
54–62 | Since there is now only a single address space mapping can this simply be deleted? | |
63 | Can then rename this to simply ASAliasRules | |
80–88 | Ditto. | |
lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp | ||
945–951 | Now there is only a single address mapping presumably this can be eliminated? |
Since there is now only a single address space mapping can this simply be deleted?