clementval added inline comments.
================ Comment at: flang/test/Fir/external-mangling.fir:36 + +// CHECK-NOUNDER: func @foo +// CHECK-NOUNDER: %{{.*}} = fir.address_of(@a) : !fir.ref<!fir.array<4xi8>> ---------------- You should check at least the character after it because here the check line would also succeed with an underscore. ================ Comment at: flang/test/Fir/external-mangling.fir:39-40 +// CHECK-NOUNDER: %{{.*}} = fir.address_of(@__BLNK__) : !fir.ref<!fir.array<4xi8>> +// CHECK-NOUNDER: fir.call @bar +// CHECK-NOUNDER: fir.call @bar2 +// CHECK-NOUNDER: fir.global common @a(dense<0> : vector<4xi8>) : !fir.array<4xi8> ---------------- Same here. ================ Comment at: flang/test/Fir/external-mangling.fir:85-88 +// CHECK-NOUNDER: func @callee +// CHECK-NOUNDER: fir.call @callee +// CHECK-NOUNDER: func @caller +// CHECK-NOUNDER: fir.call @callee ---------------- Same here. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140795/new/ https://reviews.llvm.org/D140795 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits