[clang] [KeyInstr] Complex assignment atoms (PR #134638)
Orlando Cazalet-Hyams via cfe-commits
cfe-commits at lists.llvm.org
Fri May 23 09:00:20 PDT 2025
OCHyams wrote:
Sorry I misunderstood what you were saying. Fixed, this should now have symmetry with prior patches. Expanded the test coverage. There's 4 binop assign calls that are instrumented individually which reflects the previous patch's macro usage. The expanded test coverage identified a redundant atom group nesting, which is now also fixed (and could only be fixed with the symmetrical approach, hurray). Thanks for the keen eyed review!
https://github.com/llvm/llvm-project/pull/134638
More information about the cfe-commits
mailing list