[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