[PATCH] D93082: Append ".__part.<ID>" to all basic block section symbols.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 15:41:05 PST 2020


MaskRay added a comment.

> b) The current basic block naming just adds a ".N" to the symbol name where N is some integer. This collides with how clang creates __cxx_global_var_init.N. clang creates these symbol names to call constructor functions and basic block symbol naming should not use the same style.

This reason alone is not sufficient I guess. `__part.N` greatly increases .strtab usage. Are there other benefits?


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

https://reviews.llvm.org/D93082



More information about the llvm-commits mailing list