[PATCH] D10003: Refactor: Simplify boolean conditional return statements in lib/Transforms/Vectorize

Richard via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 24 10:07:42 PDT 2015


LegalizeAdulthood added a comment.

In http://reviews.llvm.org/D10003#178196, @mzolotukhin wrote:

> Hi Richard,
>
> Thanks for cleaning this up. In the first spot I prefer the original version, but the rest look fine (and if you feel strong about the first place too, I don't mind committing the change there as well).
>
> Please mention that it's NFC in the commit message.


I don't have commit access.  Subsequent improvements to clang-tidy based on feedback from these reviews resulted in the one exception you noted no longer being generated as a simplification per LLVM/Clang coding preferences.


http://reviews.llvm.org/D10003





More information about the llvm-commits mailing list