quinnp added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:15627
+ Value *Y =
+ Builder.CreateAnd(EmitScalarExpr(E->getArg(1)),
Builder.CreateNot(Op3));
return Builder.CreateOr(X, Y);
----------------
nemanjai wrote:
> Nit: I understand that we only have one use of `E->getArg(1)`, but might as
> well initialize `Op1` as above just for consistency.
I've fixed all the instances of this.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121637/new/
https://reviews.llvm.org/D121637
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
