[PATCH] D152458: AMDGPU: Move AMDGPUAttributor run earlier

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 10:22:44 PDT 2023


arsenm created this revision.
arsenm added reviewers: AMDGPU, JonChesterfield, jdoerfert, rampitec, Pierre-vh.
Herald added subscribers: foad, okura, kerbowa, hiraditya, tpr, dstuttard, yaxunl, jvesely, kzhuravl.
Herald added a project: All.
arsenm requested review of this revision.
Herald added a subscriber: wdng.
Herald added a reviewer: sstefan1.
Herald added a project: LLVM.

Move it up with other module passes. It's a higher level optimization
that should probably be done before hacking up the IR for codegen. It
should really be done earlier than this. We could possibly move this
with other IPO passes, but we'd have to stop inferring the lack of
lds.kernel.id calls and have the LDS module pass mark functions which
don't need the ID.

      

The one test change is because that pass is relying on the backend run
of SROA (which we ideally wouldn't have).


https://reviews.llvm.org/D152458

Files:
  llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
  llvm/test/CodeGen/AMDGPU/llc-pipeline.ll
  llvm/test/CodeGen/AMDGPU/simple-indirect-call.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D152458.529666.patch
Type: text/x-patch
Size: 5604 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230608/ae882e44/attachment.bin>


More information about the llvm-commits mailing list