================ @@ -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