Wrong value for sext i1 true in BasicAliasAnalysis

Nicholas White n.j.white at gmail.com
Mon Dec 15 03:13:18 PST 2014


Hi -

Thanks for the repro. For context, I changed zext to sext on line 268 to
fix another bug (see test cases on that commit).

> I also don't understand, for example, why it is correct to truncate and
re-extend the already-computed Offset and Scale from the caller

I agree - I'll have a look tonight and see if I can come up with a proper
fix, as clearly neither zext or sext is always correct!

Nick
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141215/7a7a1658/attachment.html>


More information about the llvm-commits mailing list