[llvm] [RISCV] Enable merging of external globals by default (PR #117880)

Alex Bradbury via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 4 06:20:27 PST 2024


================
@@ -51,12 +51,9 @@ static cl::opt<cl::boolOrDefault>
     EnableGlobalMerge("riscv-enable-global-merge", cl::Hidden,
                       cl::desc("Enable the global merge pass"));
 
-static cl::opt<bool> ForceEnableGlobalMergeExternalGlobals(
-    "riscv-force-enable-global-merge-external-globals", cl::Hidden,
-    cl::init(false),
-    cl::desc(
-        "If the global merge pass is enabled, force enable global merging of "
-        "external globals (overriding any logic that might disable it)"));
+static cl::opt<bool> DisableGlobalMergeExternalGlobals(
----------------
asb wrote:

Thank you, it is - the naming "MergeExternalByDefault" should have made me go check. I've now deleted and updated the test that relies on this - thank you!

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


More information about the llvm-commits mailing list