[PATCH] D29512: [PGO] Directory name stripping in global identifier for static functions

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 3 21:50:50 PST 2017


mehdi_amini added a comment.

From the description:

> Current internal option -static-func-full-module-prefix strips all the directory path the profile counter names for static functions. The default of this option is true.

...

> This patch turns the default value for -static-func-full-module-prefix to false.

However in the diff I see:

- "static-func-full-module-prefix", cl::init(false),

+    "static-func-full-module-prefix", cl::init(true),

Which is the opposite of the description? This seems confusing to me.


https://reviews.llvm.org/D29512





More information about the llvm-commits mailing list