[llvm] Minor post-commit review changes for #130988 (NFC) (PR #156895)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 4 07:44:29 PDT 2025


================
@@ -719,12 +719,12 @@ FUNCTION_PASS_WITH_PARAMS(
     },
     parseBoundsCheckingOptions,
     "trap;rt;rt-abort;min-rt;min-rt-abort;merge;guard=N")
-    FUNCTION_PASS_WITH_PARAMS(
+FUNCTION_PASS_WITH_PARAMS(
     "expand-fp", "ExpandFpPass",
     [TM = TM](CodeGenOptLevel OL) {
       return ExpandFpPass(TM, OL);
     },
-    parseExpandFpOptions, "opt-level")
+    parseExpandFpOptions, "opt-level=N")
----------------
arsenm wrote:

This opt-level argument is strange. What do other passes do for the opt level argument? This is only used in a binary way in the pass so does it really need the level?

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


More information about the llvm-commits mailing list