pengfei added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:14742
Builder.getFastMathFlags().setAllowReassoc();
- return Builder.CreateCall(F, {Ops[0], Ops[1]});
+ Value *FAdd = Builder.CreateCall(F, {Ops[0], Ops[1]});
+ Builder.getFastMathFlags() &= (FMF);
----------------
barannikov88 wrote:
> barannikov88 wrote:
> > Could be just:
> > ```
> > return Builder.CreateCall(F, {Ops[0], Ops[1]})
> > ->getFastMathFlags()
> > .setAllowReassoc();
> > ```
> > so that you don't have to save and restore Builder's flags.
> >
> No, it couldn't be, just tried :(
Thanks for trying it :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D140467/new/
https://reviews.llvm.org/D140467
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
