[compiler-rt] [llvm] [TypeProf][InstrFDO]Omit vtable symbols in indexed profiles by default (PR #96520)

Mingming Liu via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 25 20:27:57 PDT 2024


minglotus-6 wrote:

> Perhaps make the option default to true initially?

Hmm, I'm hoping build tools or profile processing tools can pick up `--keep-vtable-symbols=false` naturally to save the need to update them individually. Let me know if there is a scenario or example we'd prefer to override the option in separate changes!
*  With `--keep-vtable-symbols=false`,  there won't be issues when merging old profiles (generated before this change) and new profiles (generated with this change).

> and with this change we won't emit either (iiuc).

Correct!

> That's a fairly small decrease, but otoh I don't see a big downside since the names are there in the raw profile which is often available.

I gathered profiles from 4 other binaries. The decrease is smaller than 1% across binaries.

|binary  |Diff|profile size before (in MB)|profile size after (in MB)|
|--------|----|---------------------------|--------------------------|
|search1 |0.8%|337                        |334                       |
|search2 |0.9%|330                        |327                       |
|database|0.5%|256.7                      |255.3                     |
|network |0.6%|98.4                       |97.8                      |




https://github.com/llvm/llvm-project/pull/96520


More information about the llvm-commits mailing list