[llvm] [SelectionDAG][X86] Fix the assertion failure in Release build after #91747 (PR #93434)

via llvm-commits llvm-commits at lists.llvm.org
Sun May 26 19:58:24 PDT 2024


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 331eb8a0047504f3ae2cdf2d6c60b93e5d0543f1 11425bd85fa77c3118b0262e22f72c96d0b7b7ad -- llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp llvm/lib/Target/X86/X86ISelLowering.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index 09284839b9..de0148d5dc 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -49253,7 +49253,8 @@ static SDValue combineX86SubCmpForFlags(SDNode *N, SDValue Flag,
     return SDValue();
 
   SDValue X = SetCC.getOperand(1);
-  // sub has two results while X only have one. DAG combine assumes the value type match.
+  // sub has two results while X only have one. DAG combine assumes the value
+  // type match.
   if (N->getNumValues() > 1)
     X = DAG.getMergeValues({N->getOperand(0), X}, SDLoc(N));
 

``````````

</details>


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


More information about the llvm-commits mailing list