[llvm] [SystemZ] Improve shouldCoalesce() for i128. (PR #74942)

via llvm-commits llvm-commits at lists.llvm.org
Sat Dec 9 09:10:51 PST 2023


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 08cb64034f17d50a660ec78ce8ea81a025b0ba71 29faa5485b5051c4a41b8d6217dd1ebc4899c41a -- llvm/lib/Target/SystemZ/SystemZRegisterInfo.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/lib/Target/SystemZ/SystemZRegisterInfo.cpp b/llvm/lib/Target/SystemZ/SystemZRegisterInfo.cpp
index e822dbf586..e2626f0b90 100644
--- a/llvm/lib/Target/SystemZ/SystemZRegisterInfo.cpp
+++ b/llvm/lib/Target/SystemZ/SystemZRegisterInfo.cpp
@@ -411,8 +411,8 @@ bool SystemZRegisterInfo::shouldCoalesce(MachineInstr *MI,
 
   // Allow cases (like PAIR128) where there is just one use of the narrow
   // source reg defined close to MI.
-  if (WideOpNo == 0 && MRI->hasOneUse(GRNarReg) &&
-      FirstMI_GRNar && FirstMI_GRNar->getParent() == MBB &&
+  if (WideOpNo == 0 && MRI->hasOneUse(GRNarReg) && FirstMI_GRNar &&
+      FirstMI_GRNar->getParent() == MBB &&
       std::distance(FirstMI_GRNar->getIterator(), MI->getIterator()) < 5)
     return true;
 

``````````

</details>


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


More information about the llvm-commits mailing list