[llvm] [X86] For minsize, use size for alignment, rather than actual alignment (PR #87003)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 16:27:32 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 d286efeb1d0d3e135901dd07535cc68aed39a758 a3a538b9d70b6f3a0a29e8ca563e0cf3d4879741 --extensions cpp -- llvm/lib/Target/X86/X86SelectionDAGInfo.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/lib/Target/X86/X86SelectionDAGInfo.cpp b/llvm/lib/Target/X86/X86SelectionDAGInfo.cpp
index 84e884fd54..bb93975771 100644
--- a/llvm/lib/Target/X86/X86SelectionDAGInfo.cpp
+++ b/llvm/lib/Target/X86/X86SelectionDAGInfo.cpp
@@ -66,9 +66,8 @@ SDValue X86SelectionDAGInfo::EmitTargetCodeForMemset(
   // The libc version is likely to be faster for these cases. It can use the
   // address value and run time information about the CPU.
   uint64_t SizeVal = ConstantSize->getZExtValue();
-  if (!AlwaysInline &&
-      (Alignment < Align(4) || !ConstantSize ||
-       SizeVal > Subtarget.getMaxInlineSizeThreshold()))
+  if (!AlwaysInline && (Alignment < Align(4) || !ConstantSize ||
+                        SizeVal > Subtarget.getMaxInlineSizeThreshold()))
     return SDValue();
 
   // If we have minsize, then don't care about the alignment.

``````````

</details>


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


More information about the llvm-commits mailing list