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

Reply via email to