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
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to