[PATCH] D97585: [InstrProfiling] Use llvm.compiler.used instead of llvm.used for ELF

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 15:49:09 PST 2021


MaskRay added a comment.

In D97585#2591568 <https://reviews.llvm.org/D97585#2591568>, @davidxl wrote:

> So to summarize, this patch is to enable compiler passes to respect comdat semantics while still enabling linker to do GC (after using GNU_RETAIN)?

The patch makes the ELF behavior the same (the metadata sections can be GCed) before or after D97448 <https://reviews.llvm.org/D97448>.

The comments and the paragraph in this patch are mainly to summarize the current situation (why we use `llvm.compiler.used/llvm.used`) :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97585



More information about the llvm-commits mailing list