stefanp accepted this revision. stefanp added a comment. This revision is now accepted and ready to land.
Thank you for adding this! Other than one minor nit I think this LGTM. Feel free to address nits on commits. ================ Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:8834 + &LosesInfo); + bool Success = (!LosesInfo && !APFloatToConvert.isDenormal()); + ---------------- nit: You can just inline this. It is only used in one place. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95458/new/ https://reviews.llvm.org/D95458 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits