[llvm] [ubsan] Pass fsanitize-skip-hot-cutoff into -fsanitize=bounds (PR #122576)

Florian Mayer via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 13 09:56:33 PST 2025


================
@@ -1315,10 +1315,18 @@ parseBoundsCheckingOptions(StringRef Params) {
     } else if (ParamName == "merge") {
       Options.Merge = true;
     } else {
-      return make_error<StringError>(
-          formatv("invalid BoundsChecking pass parameter '{0}' ", ParamName)
-              .str(),
-          inconvertibleErrorCode());
+      StringRef ParamEQ;
+      StringRef Val;
+      std::tie(ParamEQ, Val) = ParamName.split('=');
+      int8_t Id = 0;
----------------
fmayer wrote:

Why `=0`? Wouldn't it be better to leave uninitialized, because this is an output parameter? Makes it a bit more obvious I think

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


More information about the llvm-commits mailing list