mibintc added a comment. In D62731#1773854 <https://reviews.llvm.org/D62731#1773854>, @michele.scandale wrote:
> I've noticed you removed the change for `CompilerInvocation.cpp` about the > initialization of the codegen option `NoTrappingMath`. Was that an accident? I checked the old and new version of the patch and it seems like initialization of NoTrappingMath is unchanged, the definition of the option has it default to 0, and CompilerInvocation.cpp sets it like this in ParseLangArgs, was it something else you were looking at? Opts.NoTrappingMath = Args.hasArg(OPT_fno_trapping_math); Repository: rG LLVM Github Monorepo 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