[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