[Patch] Fix integer comparison in DIEInteger::BestForm

Hans Wennborg hans at chromium.org
Sun Mar 17 08:23:10 PDT 2013


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dieinteger_bestform.patch
Type: application/octet-stream
Size: 920 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130317/ffe74e74/attachment.obj>


More information about the llvm-commits mailing list