[Patch] Fix integer comparison in DIEInteger::BestForm

Eric Christopher echristo at gmail.com
Mon Mar 18 09:34:03 PDT 2013


Not surprising on the test. If you could come up with something awesome,
otherwise I think it's fine.

Thanks!

-eric


On Sun, Mar 17, 2013 at 8:23 AM, Hans Wennborg <hans at chromium.org> wrote:

> Hi all,
>
> I found the always-true comparison "(int)Int == (signed)Int" in DIE.h.
> I've attached a patch that tries to fix it. There doesn't seem to be
> any test covering this :/
>
> Please take a look!
>
> Thanks,
> Hans
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130318/181f4c30/attachment.html>


More information about the llvm-commits mailing list