<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, 7 Sep 2018 at 17:48, Tim Northover <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I looked at your diffs and you've only changed one of the functions to<br>
return SDValue(), you need to change lowerFP_TO_SINT itself too. The<br>
one with the store is just there as an optimization; if it doesn't<br>
trigger (because of your diff) then lowerFP_TO_SINT will still create<br>
a bad node afterwards.<br></blockquote><div><br>Thank you!<br><br>With that, the four bugs I found in newlib's C code are dead. I think the lrint fix should be upstreamed right away; would you mind if I credited you in the patch?<br><br>Unfortunately, we are not out of the woods yet. If memory serves me correctly, newlib has some assembly code in used for things like `_start` to bring up the C runtime, which GNU binutils accepts and Clang's assembler currently does not.<br><br>I'll go build that so I'm not talking out of my ass.<br></div><br>Cheers, Tim!<br></div></div>