[clang] [KeyInstr][Clang] For range stmt atoms (PR #134647)
Stephen Tozer via cfe-commits
cfe-commits at lists.llvm.org
Tue May 27 03:44:27 PDT 2025
================
@@ -0,0 +1,71 @@
+// RUN: %clang_cc1 -gkey-instructions %s -debug-info-kind=line-tables-only -emit-llvm -o - \
+// RUN: | FileCheck %s --implicit-check-not atomGroup --implicit-check-not atomRank
+
+// Perennial quesiton: should the inc be its own source atom or not
+// (currently it is).
+
+// FIXME: See do.c and while.c regarding cmp and cond br groups.
+
+// The stores in the setup (stores to __RANGE1, __BEGIN1, __END1) are all
+// marked as Key. Unclear whether that's desirable. Keep for now as that's
+// least risky.
----------------
SLTozer wrote:
For posterity, maybe explain what "risky" means in this context?
https://github.com/llvm/llvm-project/pull/134647
More information about the cfe-commits
mailing list