[llvm-branch-commits] [llc] Respect shouldDefaultToNewPM (PR #214572)

[Alexis Engelke via llvm-branch-commits]

aengelke wrote:

> switch llc only when GIsel is supported.

That would sound reasonable.

> clang and other frontends would still plumb options through the driver (or is it the plan to make this the master gapting option ?)

I'd still prefer to have some (non-virtual) function in TargetMachine that just "compiles this module", which internally respects shouldDefaultToNewPM and a new cl::opt that can be used to override the decision. I think the driver also supports setting LLVM command line options (e.g. -mllvm -enable-codegen-newpm).

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