[PATCH] D22868: [AArch64] Register passes so they can be run by llc

[Tim Northover via llvm-commits]

t.p.northover added a comment.

Ah, that makes sense.

> Alternatively, we could be a bit more tidy and rename all the options to "-aarch64-enable-whatever", because that describes their effect better. Do you have any preference here?


I think this option is probably simplest.

> Another inconsistency here is *where* the options are defined - a bunch of them are in AArch64TargetMachine, but that's not the case for all of the passes, e.g. AArch64BranchRelaxation defines the option in its own file.


I think we've noticed this in the past, but I can't find the thread now. It would be good to get everything consistent. Either way seems about the same to me, so you get to choose as the one doing the work!

Cheers.

Tim.


https://reviews.llvm.org/D22868