[llvm-branch-commits] [llvm] WIP: [CodeGen][NPM] Port VirtRegRewriter to NPM (PR #130564)

[via llvm-branch-commits]

github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff f20b44edd8de0001efd809147599e48966e51606 061150df047480e1bcf44460b0ff745e5d6a4b65 --extensions h,cpp -- llvm/include/llvm/CodeGen/VirtRegMap.h llvm/include/llvm/InitializePasses.h llvm/include/llvm/Passes/CodeGenPassBuilder.h llvm/lib/CodeGen/CodeGen.cpp llvm/lib/CodeGen/VirtRegMap.cpp llvm/lib/Passes/PassBuilder.cpp llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/include/llvm/CodeGen/VirtRegMap.h b/llvm/include/llvm/CodeGen/VirtRegMap.h
index e74131545f..f5fba0d654 100644
--- a/llvm/include/llvm/CodeGen/VirtRegMap.h
+++ b/llvm/include/llvm/CodeGen/VirtRegMap.h
@@ -238,8 +238,10 @@ public:
 
 class VirtRegRewriterPass : public PassInfoMixin<VirtRegRewriterPass> {
   bool ClearVirtRegs = true;
+
 public:
-  VirtRegRewriterPass(bool ClearVirtRegs = true) : ClearVirtRegs(ClearVirtRegs) {}
+  VirtRegRewriterPass(bool ClearVirtRegs = true)
+      : ClearVirtRegs(ClearVirtRegs) {}
   PreservedAnalyses run(MachineFunction &MF,
                         MachineFunctionAnalysisManager &MFAM);
 
diff --git a/llvm/lib/CodeGen/VirtRegMap.cpp b/llvm/lib/CodeGen/VirtRegMap.cpp
index 218b52ab7a..bf10fbe48b 100644
--- a/llvm/lib/CodeGen/VirtRegMap.cpp
+++ b/llvm/lib/CodeGen/VirtRegMap.cpp
@@ -230,16 +230,15 @@ public:
       : Indexes(Indexes), LIS(LIS), LRM(LRM), VRM(VRM), DebugVars(DebugVars),
         ClearVirtRegs(ClearVirtRegs) {}
 
-  bool run(MachineFunction&);
-
+  bool run(MachineFunction &);
 };
 
 class VirtRegRewriterLegacy : public MachineFunctionPass {
 public:
   static char ID;
   bool ClearVirtRegs;
-  VirtRegRewriterLegacy(bool ClearVirtRegs = true) :
-    MachineFunctionPass(ID), ClearVirtRegs(ClearVirtRegs) {}
+  VirtRegRewriterLegacy(bool ClearVirtRegs = true)
+      : MachineFunctionPass(ID), ClearVirtRegs(ClearVirtRegs) {}
 
   void getAnalysisUsage(AnalysisUsage &AU) const override;
 
@@ -302,13 +301,15 @@ bool VirtRegRewriterLegacy::runOnMachineFunction(MachineFunction &MF) {
   return R.run(MF);
 }
 
-PreservedAnalyses VirtRegRewriterPass::run(MachineFunction &MF,
-  MachineFunctionAnalysisManager &MFAM) {
+PreservedAnalyses
+VirtRegRewriterPass::run(MachineFunction &MF,
+                         MachineFunctionAnalysisManager &MFAM) {
   VirtRegMap &VRM = MFAM.getResult<VirtRegMapAnalysis>(MF);
   LiveIntervals &LIS = MFAM.getResult<LiveIntervalsAnalysis>(MF);
   LiveRegMatrix &LRM = MFAM.getResult<LiveRegMatrixAnalysis>(MF);
   SlotIndexes &Indexes = MFAM.getResult<SlotIndexesAnalysis>(MF);
-  LiveDebugVariables &DebugVars = MFAM.getResult<LiveDebugVariablesAnalysis>(MF);
+  LiveDebugVariables &DebugVars =
+      MFAM.getResult<LiveDebugVariablesAnalysis>(MF);
 
   VirtRegRewriter R(ClearVirtRegs, &Indexes, &LIS, &LRM, &VRM, &DebugVars);
   if (!R.run(MF))
@@ -771,7 +772,8 @@ void VirtRegRewriter::rewrite() {
   RewriteRegs.clear();
 }
 
-void VirtRegRewriterPass::printPipeline(raw_ostream &OS, function_ref<StringRef(StringRef)>) const {
+void VirtRegRewriterPass::printPipeline(
+    raw_ostream &OS, function_ref<StringRef(StringRef)>) const {
   OS << "virt-reg-rewriter<";
   if (!ClearVirtRegs)
     OS << "no-";

``````````

</details>


https://github.com/llvm/llvm-project/pull/130564