[PATCH] D14792: Have 'optnone' respect -fast-isel=false

Paul Robinson via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 19 19:44:02 PST 2015


probinson added a comment.

In http://reviews.llvm.org/D14792#293572, @echristo wrote:

> Just make it an enum ala FloatABI with a Default, SelectionDAG, FastISel
>  and use that everywhere?
>
> -eric


"Everywhere" is exactly two places, hardly seems worth the trouble.  Maybe rename the flag to "O0UsesFastISel" instead? which then can be used directly in both LLVMTargetMachine and OptLevelChanger with obviously consistent semantics.


http://reviews.llvm.org/D14792





More information about the llvm-commits mailing list