[llvm-branch-commits] [clang] [KeyInstr][Clang] Assignment atom group (PR #134637)

[Jeremy Morse via llvm-branch-commits]

================
@@ -5985,6 +5985,15 @@ LValue CodeGenFunction::EmitBinaryOperatorLValue(const BinaryOperator *E) {
 
----------------
jmorse wrote:

I wonder whether the comma operator (six or seven lines above here) needs instrumenting -- technically if either lhs/rhs of the comma is an assignment, I guess it'll come back through this function and be given a separate group?

https://github.com/llvm/llvm-project/pull/134637