https://llvm.org/bugs/show_bug.cgi?id=23038

James Molloy <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from James Molloy <[email protected]> ---
Hi Nick,

Thanks for raising this. It was a stupid error about the polarity of an
"isUnsigned" parameter (I thought true was signed, but true is unsigned. Sigh.)

I've added a new regression test to pick this up and also added a comment to
clarify the polarity.

I've reapplied in r233370, I hope this revision doesn't give you any more
problems. If it does, perhaps it might be easier for change tracking to just
flip the default of the -float-to-int flag in PassManagerBuilder to false. That
way my changes are tracked and can be post-commit reviewed if people want to.

Cheers, and sorry for any inconvenience,

James

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
LLVMbugs mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs

Reply via email to