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

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 27 09:48:48 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(
----------------
topperc wrote:

Is this option redundant with the `global-merge-on-external` option in the pass?

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


More information about the llvm-commits mailing list