================
@@ -722,6 +713,9 @@ struct MCDCCoverageBuilder {
       return I->second;
   }
 
+  /// Return the LHS Decision ([0,0] if not set).
+  const DecisionIDPair back() const { return DecisionStack.back(); }
----------------
chapuni wrote:

This may return byref since `DecisionStack.back()` returns byref. I think this 
is consistent.
The callee (visitBin) may take an option to copy values instead of holding the 
ref.

I guess the change doesn't have any big impacts for generated code. I suggest 
just for the consistency.

https://github.com/llvm/llvm-project/pull/78202
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to