Skip to content
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

[SystemZ] Add pass initialization for CopyPhysRegs pass #132912

Merged

Conversation

dominik-steenken
Copy link
Contributor

This commit adds an initialization to SystemZTargetMachine for the SystemZCopyPhysRegs pass that was thus far missing. This will enable, e.g., to run tests for that pass via llc --run-pass=systemz-copy-physregs.

This commit adds an initialization to `SystemZTargetMachine` for the
`SystemZCopyPhysRegs` pass that was thus far missing. This will enable,
e.g., to run tests for that pass via `llc --run-pass=systemz-copy-physregs`.
@llvmbot
Copy link
Member

llvmbot commented Mar 25, 2025

@llvm/pr-subscribers-backend-systemz

Author: Dominik Steenken (dominik-steenken)

Changes

This commit adds an initialization to SystemZTargetMachine for the SystemZCopyPhysRegs pass that was thus far missing. This will enable, e.g., to run tests for that pass via llc --run-pass=systemz-copy-physregs.


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

1 Files Affected:

  • (modified) llvm/lib/Target/SystemZ/SystemZTargetMachine.cpp (+1)
diff --git a/llvm/lib/Target/SystemZ/SystemZTargetMachine.cpp b/llvm/lib/Target/SystemZ/SystemZTargetMachine.cpp
index 092515ee197a8..6300fb72990d0 100644
--- a/llvm/lib/Target/SystemZ/SystemZTargetMachine.cpp
+++ b/llvm/lib/Target/SystemZ/SystemZTargetMachine.cpp
@@ -48,6 +48,7 @@ extern "C" LLVM_EXTERNAL_VISIBILITY void LLVMInitializeSystemZTarget() {
   initializeSystemZPostRewritePass(PR);
   initializeSystemZTDCPassPass(PR);
   initializeSystemZDAGToDAGISelLegacyPass(PR);
+  initializeSystemZCopyPhysRegsPass(PR);
 }
 
 static std::string computeDataLayout(const Triple &TT) {

Copy link
Member

@uweigand uweigand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@uweigand uweigand merged commit 2a10e3d into llvm:main Mar 25, 2025
13 checks passed
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