[clang] [llvm] [ubsan] Parse and use <cutoffs[0,1,2]=70000;cutoffs[5,6,8]=90000> in LowerAllowCheckPass (PR #124211)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 16:52:13 PST 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 ec15b242505a46ea7d195a6520fb869a80a2cd10 7b6b0166835faff53665321342b47b0887ee99a6 --extensions cpp,h -- clang/lib/Frontend/CompilerInvocation.cpp llvm/include/llvm/Transforms/Instrumentation/LowerAllowCheckPass.h llvm/lib/Passes/PassBuilder.cpp llvm/lib/Transforms/Instrumentation/BoundsChecking.cpp llvm/lib/Transforms/Instrumentation/LowerAllowCheckPass.cpp
``````````

</details>

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

``````````diff
diff --git a/llvm/lib/Passes/PassBuilder.cpp b/llvm/lib/Passes/PassBuilder.cpp
index abe97d8d3d..bc4aba48a4 100644
--- a/llvm/lib/Passes/PassBuilder.cpp
+++ b/llvm/lib/Passes/PassBuilder.cpp
@@ -850,14 +850,14 @@ parseLowerAllowCheckPassOptions(StringRef Params) {
                 .str(),
             inconvertibleErrorCode());
 
-      if (!IndicesStr.consume_front("cutoffs[") || !IndicesStr.consume_back("]")
-          || IndicesStr == "")
-          return make_error<StringError>(
-              formatv("invalid LowerAllowCheck pass index parameter '{0}' "
-                      "({1})",
-                      IndicesStr, CutoffStr)
-                  .str(),
-              inconvertibleErrorCode());
+      if (!IndicesStr.consume_front("cutoffs[") ||
+          !IndicesStr.consume_back("]") || IndicesStr == "")
+        return make_error<StringError>(
+            formatv("invalid LowerAllowCheck pass index parameter '{0}' "
+                    "({1})",
+                    IndicesStr, CutoffStr)
+                .str(),
+            inconvertibleErrorCode());
 
       while (IndicesStr != "") {
         StringRef firstIndexStr;

``````````

</details>


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


More information about the llvm-commits mailing list