[llvm] Makslevental/unison tblgen (PR #150084)

via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 22 11:37:44 PDT 2025


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 HEAD~1 HEAD --extensions h,cpp -- llvm/include/llvm/TableGen/Unison.h llvm/lib/CodeGen/UnisonMIRPrepare.cpp llvm/lib/TableGen/Unison.cpp llvm/include/llvm/CodeGen/Passes.h llvm/include/llvm/InitializePasses.h llvm/lib/CodeGen/CodeGen.cpp llvm/lib/CodeGen/MIRParser/MILexer.cpp llvm/lib/CodeGen/MIRParser/MILexer.h llvm/lib/CodeGen/MIRParser/MIParser.cpp llvm/lib/CodeGen/MIRPrinter.cpp llvm/lib/CodeGen/MIRPrintingPass.cpp llvm/utils/TableGen/Basic/TableGen.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/include/llvm/InitializePasses.h b/llvm/include/llvm/InitializePasses.h
index da0ff22ac..3a2bcc470 100644
--- a/llvm/include/llvm/InitializePasses.h
+++ b/llvm/include/llvm/InitializePasses.h
@@ -340,7 +340,7 @@ LLVM_ABI void initializeWindowsSecureHotPatchingPass(PassRegistry &);
 LLVM_ABI void initializeWinEHPreparePass(PassRegistry &);
 LLVM_ABI void initializeWriteBitcodePassPass(PassRegistry &);
 LLVM_ABI void initializeXRayInstrumentationLegacyPass(PassRegistry &);
-LLVM_ABI void initializeUnisonMIRPreparePass(PassRegistry&);
+LLVM_ABI void initializeUnisonMIRPreparePass(PassRegistry &);
 
 } // end namespace llvm
 
diff --git a/llvm/lib/CodeGen/MIRParser/MIParser.cpp b/llvm/lib/CodeGen/MIRParser/MIParser.cpp
index 90e037bfc..4c93a6204 100644
--- a/llvm/lib/CodeGen/MIRParser/MIParser.cpp
+++ b/llvm/lib/CodeGen/MIRParser/MIParser.cpp
@@ -755,8 +755,9 @@ bool MIParser::parseBasicBlockDefinition(
         break;
       case MIToken::kw_freq:
         // Unison MIR style extension: basic block execution frequency.
-          lex();
-        if (Token.isNot(MIToken::IntegerLiteral) || Token.integerValue().isSigned())
+        lex();
+        if (Token.isNot(MIToken::IntegerLiteral) ||
+            Token.integerValue().isSigned())
           return error("expected an integer literal");
         lex();
         break;
diff --git a/llvm/lib/CodeGen/MIRPrinter.cpp b/llvm/lib/CodeGen/MIRPrinter.cpp
index e9f8a3839..822770826 100644
--- a/llvm/lib/CodeGen/MIRPrinter.cpp
+++ b/llvm/lib/CodeGen/MIRPrinter.cpp
@@ -721,13 +721,11 @@ void printMBB(raw_ostream &OS, MFPrintState &State,
       OS << LS << printMBBReference(**I);
       // The Unison style uses a simpler formatting of the probabilities.
       if (UnisonMIR && (!SimplifyMIR || !canPredictProbs))
-        OS << '('
-           << MBB.getSuccProbability(I).scale(100)
-           << ')';
+        OS << '(' << MBB.getSuccProbability(I).scale(100) << ')';
       else // Intentional indention to reduce merge conflicts.
-      if (!SimplifyMIR || !canPredictProbs)
-        OS << format("(0x%08" PRIx32 ")",
-                     MBB.getSuccProbability(I).getNumerator());
+        if (!SimplifyMIR || !canPredictProbs)
+          OS << format("(0x%08" PRIx32 ")",
+                       MBB.getSuccProbability(I).getNumerator());
     }
     OS << "\n";
     HasLineAttributes = true;
diff --git a/llvm/lib/CodeGen/MIRPrintingPass.cpp b/llvm/lib/CodeGen/MIRPrintingPass.cpp
index eff731521..f76c1704a 100644
--- a/llvm/lib/CodeGen/MIRPrintingPass.cpp
+++ b/llvm/lib/CodeGen/MIRPrintingPass.cpp
@@ -14,11 +14,11 @@
 #include "llvm/CodeGen/MIRPrinter.h"
 #include "llvm/CodeGen/MachineFunctionPass.h"
 #include "llvm/CodeGen/MachineModuleInfo.h"
-#include "llvm/CodeGen/UnisonMIRPrepare.h"
 #include "llvm/CodeGen/Passes.h"
+#include "llvm/CodeGen/UnisonMIRPrepare.h"
 #include "llvm/IR/Function.h"
-#include "llvm/Support/CommandLine.h"
 #include "llvm/InitializePasses.h"
+#include "llvm/Support/CommandLine.h"
 
 using namespace llvm;
 
diff --git a/llvm/lib/CodeGen/UnisonMIRPrepare.cpp b/llvm/lib/CodeGen/UnisonMIRPrepare.cpp
index e01b69cac..246019578 100644
--- a/llvm/lib/CodeGen/UnisonMIRPrepare.cpp
+++ b/llvm/lib/CodeGen/UnisonMIRPrepare.cpp
@@ -39,15 +39,14 @@
 //===----------------------------------------------------------------------===//
 
 #include "llvm/CodeGen/UnisonMIRPrepare.h"
-#include "llvm/Pass.h"
 #include "llvm/CodeGen/MachineBlockFrequencyInfo.h"
 #include "llvm/CodeGen/MachineInstrBuilder.h"
 #include "llvm/CodeGen/TargetInstrInfo.h"
-#include "llvm/CodeGen/MachineBlockFrequencyInfo.h"
 #include "llvm/CodeGen/TargetSubtargetInfo.h"
 #include "llvm/IR/Constants.h"
-#include "llvm/InitializePasses.h"
 #include "llvm/IR/MDBuilder.h"
+#include "llvm/InitializePasses.h"
+#include "llvm/Pass.h"
 
 using namespace llvm;
 
@@ -66,10 +65,9 @@ MDNode *createMDTaggedTuple(MachineFunction &MF, std::string Tag,
                             uint64_t Val) {
   LLVMContext &Context = MF.getFunction().getContext();
   MDBuilder Builder(Context);
-  return MDNode::get(Context,
-                     {Builder.createString(Tag),
-                      Builder.createConstant(
-                          ConstantInt::get(Type::getInt64Ty(Context), Val))});
+  return MDNode::get(Context, {Builder.createString(Tag),
+                               Builder.createConstant(ConstantInt::get(
+                                   Type::getInt64Ty(Context), Val))});
 }
 
 UnisonMIRPrepare::UnisonMIRPrepare() : MachineFunctionPass(ID) {

``````````

</details>


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


More information about the llvm-commits mailing list