[PATCH] D106193: [CSSPGO] Turn on unique linkage name by default for pseudo probe.

[Hongtao Yu via Phabricator via cfe-commits]

hoy added inline comments.


================
Comment at: clang/test/Driver/pseudo-probe.c:4
 // RUN: %clang -### -fpseudo-probe-for-profiling -fdebug-info-for-profiling %s 2>&1 | FileCheck %s --check-prefix=CONFLICT
+// RUN: %clang -### -fpseudo-probe-for-profiling -funique-internal-linkage-names %s 2>&1 | FileCheck %s --check-prefix=YESPROBE
+// RUN: %clang -### -fpseudo-probe-for-profiling -fno-unique-internal-linkage-names %s 2>&1 | FileCheck %s --check-prefix=NONAME
----------------
wenlei wrote:
> hoy wrote:
> > wenlei wrote:
> > > how do we test unique linkage name is implied with probe on?
> > These two test lines below corresponding to line 1 in the test, which show that -funique-internal-linkage-names is passed to cc1.
> > 
> > ```
> > 
> > // YESPROBE: -fpseudo-probe-for-profiling
> > // YESPROBE: -funique-internal-linkage-names
> > ```
> > 
> > The change being made here is a driver change, so a driver test should be good.
> The change is about making `-funique-internal-linkage-names` implied, but the test specified that switch explicitly. I was asking how to test the "implicit" part? Not sure what line 4 is testing wrt to this change. 
Line 1 is the -funique-internal-linkage-names switch , and we are checking `YESPROBE` for it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106193