[PATCH] D21299: [Codegen Prepare] Swap commutative binops before splitting branch condition.

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 11 14:31:33 PDT 2016


rengolin added a comment.

If all cores that have FeaturePredictableSelectIsExpensive can also have the new flag, and it makes sense, it could be coalesced into a single flag?

It does look a bit redundant as it is, but I haven't really looked closely how the two features are linked.

@t.p.northover,any comments?


http://reviews.llvm.org/D21299





More information about the llvm-commits mailing list