[PATCH] D34220: [AArch64] Prefer B.cond to CBZ/CBNZ/TBZ/TBNZ when NZCV flags can be set for "free"
Chad Rosier via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 15 11:13:28 PDT 2017
mcrosier added a comment.
In https://reviews.llvm.org/D34220#781363, @meadori wrote:
> RE: the feature flag statement. Unless we know that this is a win all supported sub-targets (micro-architectures), then it seems like a feature flag would be the way to go, right?
Yes, I think this is reasonable. I left it off initially assuming it would be easier for other sub-target owners to test, but if the consensus is this should be sub-target specific, I'll add the feature flag accordingly.
https://reviews.llvm.org/D34220
More information about the llvm-commits
mailing list