[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