[PATCH] D38637: [InstSimplify] don't let poison inhibit an easy fold
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 6 12:20:41 PDT 2017
efriedma added a comment.
If I understand correctly, the reason computeKnownBits can't handle this is that it doesn't know what to do with a known poison value? We could just solve the issue in computeKnownBits: currently, it says there are no known bits when it detects a shift overflow, but it could just say, for example, that all the bits are known zero (since the result of computeKnownBits is only meaningful if the value isn't poison).
https://reviews.llvm.org/D38637
More information about the llvm-commits
mailing list