Skip to content

Commit 98e8b99

Browse files
committed
Clean up alias-analysis-2.fir comments
Don't introduce T1 and T2 comments as I'm not sure they matter so much going forward. As for T3, make it clear what T4 references.
1 parent e094ffb commit 98e8b99

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

flang/test/Analysis/AliasAnalysis/alias-analysis-2.fir

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
// p1.addr and p2.addr result from 2 different allocas
88
// They cannot physically alias
9-
// T1: CHECK-DAG: p1.addr#0 <-> p2.addr#0: NoAlias
9+
// CHECK-DAG: p1.addr#0 <-> p2.addr#0: NoAlias
1010

1111
// p1.addr is the address returned by an alloca. It does not have a target
1212
// attribute, and it is not the address retrieved from a pointer. It cannot
@@ -31,7 +31,7 @@
3131
// All arguments are either pointers or targets
3232
// The address *in* a local pointer may alias the address of a target
3333
// argument, but it does not alias the address *of* a pointer argument.
34-
// T2: CHECK-DAG: boxp1.addr#0 <-> func.region0#0: MayAlias
34+
// CHECK-DAG: boxp1.addr#0 <-> func.region0#0: MayAlias
3535
// CHECK-DAG: boxp1.addr#0 <-> func.region0#1: MayAlias
3636
// CHECK-DAG: boxp1.addr#0 <-> func.region0#2: NoAlias
3737

@@ -57,7 +57,7 @@
5757

5858
// The addresses stored in two different pointers can alias, even if one has no
5959
// box. In this program, they happen to be the same address.
60-
// T4:
60+
// T4 from <https://github.com/llvm/llvm-project/pull/117785#discussion_r1924348480>.
6161
// CHECK-DAG: p1.tgt#0 <-> boxp1.addr#0: MayAlias
6262

6363
func.func @_QFPtest(%arg0: !fir.ref<f32> {fir.bindc_name = "v1", fir.target}, %arg1: !fir.ref<f32> {fir.bindc_name = "v2", fir.target}, %arg2: !fir.ref<!fir.box<!fir.ptr<f32>>> ) attributes {test.ptr = "func"} {

0 commit comments

Comments
 (0)