[PATCH] Refacotr; Simplify boolean expressions in AArch64 target

Alexander Kornienko alexfh at google.com
Sun Mar 22 15:20:13 PDT 2015


In http://reviews.llvm.org/D8525#144898, @rengolin wrote:

> On all three instances, please, keep the parenthesis, to avoid future problems with order of operators.


Do you find the order of operations unclear as it is now or do you expect confusion later if the code is changed?

I personally find `return a < b;` nicer and more consistent with existing code than `return (a < b);`.


http://reviews.llvm.org/D8525

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list