[PATCH] D108943: Fine grain control over some symbol visibility

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 30 16:51:53 PDT 2021


MaskRay added a comment.

(Background:  -fvisibility-inlines-hidden is already the default.)

The description says:

> Setting -fvisibility=hidden when compiling Target libs has the advantage of not being intrusive on the codebase, but it also sets the visibility of all functions within header-only component like ADT.

However I cannot connect this with the effect of this patch. With this patch, what exact configurations are now working? Can you show the detailed cflags?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108943



More information about the llvm-commits mailing list