Skip to content

[AMDGPU][NPM] Cleanup AMDGPUPassRegistry.def #130071

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

optimisan
Copy link
Contributor

@optimisan optimisan commented Mar 6, 2025

Finishing up AMDGPU specific passes. Only ones remaining are assembly printer, virt reg rewriter and PEI.

@llvmbot
Copy link
Member

llvmbot commented Mar 6, 2025

@llvm/pr-subscribers-backend-amdgpu

Author: Akshat Oke (optimisan)

Changes

Finishing up AMDGPU specific passes. Only ones remaining are assembly printer, virt reg rewriter and PEI.


Full diff: https://github.com/llvm/llvm-project/pull/130071.diff

3 Files Affected:

  • (modified) llvm/lib/Target/AMDGPU/AMDGPUPassRegistry.def (+1-7)
  • (modified) llvm/lib/Target/AMDGPU/AMDGPUPreloadKernArgProlog.cpp (+1-1)
  • (modified) llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp (+1)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUPassRegistry.def b/llvm/lib/Target/AMDGPU/AMDGPUPassRegistry.def
index f14499d0d3146..ad2f3fc29077c 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUPassRegistry.def
+++ b/llvm/lib/Target/AMDGPU/AMDGPUPassRegistry.def
@@ -102,6 +102,7 @@ MACHINE_FUNCTION_PASS("amdgpu-pre-ra-long-branch-reg", GCNPreRALongBranchRegPass
 MACHINE_FUNCTION_PASS("amdgpu-rewrite-partial-reg-uses", GCNRewritePartialRegUsesPass())
 MACHINE_FUNCTION_PASS("amdgpu-set-wave-priority", AMDGPUSetWavePriorityPass())
 MACHINE_FUNCTION_PASS("amdgpu-pre-ra-optimizations", GCNPreRAOptimizationsPass())
+MACHINE_FUNCTION_PASS("amdgpu-preload-kern-arg-prolog", AMDGPUPreloadKernArgPrologPass())
 MACHINE_FUNCTION_PASS("amdgpu-nsa-reassign", GCNNSAReassignPass())
 MACHINE_FUNCTION_PASS("gcn-dpp-combine", GCNDPPCombinePass())
 MACHINE_FUNCTION_PASS("gcn-create-vopd", GCNCreateVOPDPass())
@@ -131,13 +132,6 @@ MACHINE_FUNCTION_PASS("si-wqm", SIWholeQuadModePass())
 #undef MACHINE_FUNCTION_PASS
 
 #define DUMMY_MACHINE_FUNCTION_PASS(NAME, CREATE_PASS)
-DUMMY_MACHINE_FUNCTION_PASS("amdgpu-pre-ra-optimizations", GCNPreRAOptimizationsPass())
-DUMMY_MACHINE_FUNCTION_PASS("amdgpu-rewrite-partial-reg-uses", GCNRewritePartialRegUsesPass())
-
-// TODO: Move amdgpu-preload-kern-arg-prolog to MACHINE_FUNCTION_PASS since it
-// already exists.
-DUMMY_MACHINE_FUNCTION_PASS("amdgpu-preload-kern-arg-prolog", AMDGPUPreloadKernArgPrologPass())
-
 // Global ISel passes
 DUMMY_MACHINE_FUNCTION_PASS("amdgpu-prelegalizer-combiner", AMDGPUPreLegalizerCombinerPass())
 DUMMY_MACHINE_FUNCTION_PASS("amdgpu-postlegalizer-combiner", AMDGPUPostLegalizerCombinerPass())
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUPreloadKernArgProlog.cpp b/llvm/lib/Target/AMDGPU/AMDGPUPreloadKernArgProlog.cpp
index b3a2139dfd24e..40094518dce0a 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUPreloadKernArgProlog.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUPreloadKernArgProlog.cpp
@@ -207,5 +207,5 @@ AMDGPUPreloadKernArgPrologPass::run(MachineFunction &MF,
   if (!AMDGPUPreloadKernArgProlog(MF).run())
     return PreservedAnalyses::all();
 
-  return PreservedAnalyses::none();
+  return getMachineFunctionPassPreservedAnalyses();
 }
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp b/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
index f380ddd03957f..a71766f2fd012 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
@@ -24,6 +24,7 @@
 #include "AMDGPUMacroFusion.h"
 #include "AMDGPUOpenCLEnqueuedBlockLowering.h"
 #include "AMDGPUPerfHintAnalysis.h"
+#include "AMDGPUPreloadKernArgProlog.h"
 #include "AMDGPURemoveIncompatibleFunctions.h"
 #include "AMDGPUSplitModule.h"
 #include "AMDGPUTargetObjectFile.h"

@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch 2 times, most recently from dc9e5f0 to d4ed8c0 Compare March 7, 2025 10:16
@optimisan optimisan force-pushed the users/optimisan/03-06-_amdgpu_npm_cleanup_amdgpupassregistry.def branch from 462a055 to e281c45 Compare March 7, 2025 10:17
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from d4ed8c0 to 92a1774 Compare March 10, 2025 04:45
@optimisan optimisan force-pushed the users/optimisan/03-06-_amdgpu_npm_cleanup_amdgpupassregistry.def branch from e281c45 to 96b27b6 Compare March 10, 2025 04:45
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from 92a1774 to 94618bb Compare March 10, 2025 05:22
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from 94618bb to 9aaee4f Compare March 11, 2025 09:58
@optimisan optimisan force-pushed the users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm branch from 9aaee4f to e349243 Compare April 15, 2025 04:32
Base automatically changed from users/optimisan/03-06-_codegen_npm_port_stackframelayoutanalysispass_to_npm to main April 15, 2025 07:07
@optimisan optimisan force-pushed the users/optimisan/03-06-_amdgpu_npm_cleanup_amdgpupassregistry.def branch from 96b27b6 to e4ef948 Compare April 16, 2025 05:19
@optimisan optimisan merged commit 8408222 into main Apr 17, 2025
9 of 11 checks passed
@optimisan optimisan deleted the users/optimisan/03-06-_amdgpu_npm_cleanup_amdgpupassregistry.def branch April 17, 2025 04:38
var-const pushed a commit to ldionne/llvm-project that referenced this pull request Apr 17, 2025
Finishing up AMDGPU specific passes. Only ones remaining are assembly
printer, virt reg rewriter and PEI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants