mibintc added a comment. The incorrect code is actually in the IRBuilder which is part of a different patch...
In D62731#1750312 <https://reviews.llvm.org/D62731#1750312>, @echristo wrote: > In D62731#1749916 <https://reviews.llvm.org/D62731#1749916>, @mibintc wrote: > > > Thanks I see it, I'm working on a patch. Previously there was no support > > for frounding-math (unimplemented). This patch enables the option. In the > > IR builder, there's a call to a runtime function in the exception handler > > which is unexpectedly null. I start by adding a null pointer check. > > > Had a crash on valid here for days, let's revert and then get a fix when > recommitting. I'll respond to the thread when reverting. Thanks :) Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62731/new/ https://reviews.llvm.org/D62731 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits