[clang] [KeyInstr] Complex assignment atoms (PR #134638)

https://github.com/OCHyams closed https://github.com/llvm/llvm-project/pull/134638 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [KeyInstr] Complex assignment atoms (PR #134638)

https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/134638 >From fa1c06148d4f79e9464d92ab5c26d9a22c5bf41d Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Thu, 3 Apr 2025 13:36:59 +0100 Subject: [PATCH 01/10] [KeyInstr] Complex assignment atoms This patch is

[clang] [KeyInstr] Complex assignment atoms (PR #134638)

https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/134638 >From fa1c06148d4f79e9464d92ab5c26d9a22c5bf41d Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Thu, 3 Apr 2025 13:36:59 +0100 Subject: [PATCH 1/9] [KeyInstr] Complex assignment atoms This patch is pa

[clang] [KeyInstr] Complex assignment atoms (PR #134638)

https://github.com/jmorse approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/134638 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [KeyInstr] Complex assignment atoms (PR #134638)

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

[clang] [KeyInstr] Complex assignment atoms (PR #134638)

https://github.com/OCHyams updated https://github.com/llvm/llvm-project/pull/134638 >From fa1c06148d4f79e9464d92ab5c26d9a22c5bf41d Mon Sep 17 00:00:00 2001 From: Orlando Cazalet-Hyams Date: Thu, 3 Apr 2025 13:36:59 +0100 Subject: [PATCH 1/8] [KeyInstr] Complex assignment atoms This patch is pa

[clang] [KeyInstr] Complex assignment atoms (PR #134638)

OCHyams wrote: Sorry to all that were added... grappling with messed-up graphite internal state, GitHub enjoys adding reviewers for me when I change base. https://github.com/llvm/llvm-project/pull/134638 ___ cfe-commits mailing list cfe-commits@lists.

[clang] [KeyInstr] Complex assignment atoms (PR #134638)

https://github.com/OCHyams edited https://github.com/llvm/llvm-project/pull/134638 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits