[PATCH] D20845: DAGCombiner: Fix broken size check in isAlias

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue May 31 17:22:16 PDT 2016


hfinkel added a subscriber: hfinkel.
hfinkel accepted this revision.
hfinkel added a reviewer: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM

Can you open a PR against Clang here? We should warn on this, the term (the boolean, 1 or 0, shifted right by three) is always zero. I assume the general warning here is considering the boolean to be a byte in size for some reason.


http://reviews.llvm.org/D20845





More information about the llvm-commits mailing list