================
@@ -78,7 +78,7 @@ struct BinOpInfo {
class ScalarExprEmitter : public StmtVisitor<ScalarExprEmitter, mlir::Value> {
CIRGenFunction &cgf;
CIRGenBuilderTy &builder;
- bool ignoreResultAssign;
+ bool ignoreResultAssign = false;
----------------
andykaylor wrote:
We had a discussion about this early on in the upstreaming, and we decided not
to upstream `TestAndClearIgnoreResultAssign` specifically because it was so
often obscuring the fact that we just wanted to set it to false. I tried to
find the discussion, but so far with no luck. We're using `std::exchange` in
the few places where we actually want the value before clearing it.
https://github.com/llvm/llvm-project/pull/166118
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
