[PATCH] D96109: Refactor implementation of -funique-internal-linkage-names.

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added inline comments.


================
Comment at: clang/test/CodeGen/unique-internal-linkage-names.cpp:57
 // UNIQUE: @_ZN12_GLOBAL__N_16anon_mE.__uniq.{{[0-9]+}} = internal global
-// UNIQUE: define internal i32 @_ZL3foov.__uniq.{{[0-9]+}}()
+// UNIQUE: define internal i32 @_ZL3foov.__uniq.{{[0-9]+}}() [[ATTR:#[0-9]+]]
 // UNIQUE: define internal i32 @_ZN12_GLOBAL__N_14getMEv.__uniq.{{[0-9]+}}
----------------
https://llvm.org/docs/CommandGuide/FileCheck.html#filecheck-numeric-substitution-blocks
My suggestion was `[[#ATTR:]]`, which is shorter.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96109/new/

https://reviews.llvm.org/D96109