<div dir="ltr"><a href="https://reviews.llvm.org/D92433">https://reviews.llvm.org/D92433</a> for an example of changing LTO pass manager defaults (not ready to submit yet, need to fix a couple things first)<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 1, 2020 at 12:34 PM Arthur Eubanks <<a href="mailto:aeubanks@google.com">aeubanks@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>The ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER CMake flag currently only affects Clang. It should probably also change all other uses of pass managers where possible.  <br></div><div></div><div><br></div><div>There are a couple of uses inside LLD for LTO which already have new/legacy PM flags and should probably look at 

ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER to determine the default. <a href="https://github.com/llvm/llvm-project/blob/1314a4938fba865412598b7227cb4657d59cd8bc/lld/wasm/Driver.cpp#L382" target="_blank">Some</a> <a href="https://github.com/llvm/llvm-project/blob/1314a4938fba865412598b7227cb4657d59cd8bc/llvm/include/llvm/LTO/Config.h#L53" target="_blank">examples</a>.</div><div><br></div><div>Also at some point in the future when check-llvm has been fixed to work with opt's -enable-new-pm flag by default, that should also be dependent upon ENABLE_EXPERIMENTAL_NEW_PASS_MANAGER.</div><div><br></div><div>Any objections?</div></div>
</blockquote></div>