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?