[PATCH] D19353: [GVN] Replace an inverted comparison with a logical not

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 21 08:19:36 PDT 2016


reames added a subscriber: reames.
reames added a comment.

As a canonicalization, this makes sense, but I'm worried it will lead to bad code generation.  Having to materialize a flag into a register to negate it seems potentially problematic.  Do we already have code in the backend (CGP?) which undoes this transformation?  If not, I think we're likely to need it.


http://reviews.llvm.org/D19353





More information about the llvm-commits mailing list