kpn marked 2 inline comments as done.
kpn added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:357
+
+ if (ConstrainedIntrinsicID != Intrinsic::not_intrinsic &&
+ CGF.Builder.getIsFPConstrained()) {
----------------
rjmccall wrote:
> I don't see any situation where `not_intrinsic` is passed in here; why all
> these checks?
I tend to be conservative, and I'm not good at predicting the future. I can
remove the checks easily in the next round.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:2302
+ return RValue::get(Builder.CreateCall(F, {Base, Exponent}));
+ }
}
----------------
rjmccall wrote:
> What can this not use the standard path?
I'll take another look. It wasn't using the standard path before so I made my
added code match. If it can use the standard path in both cases then maybe it
should?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70256/new/
https://reviews.llvm.org/D70256
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
