This is an archive of the discontinued LLVM Phabricator instance.

[LTO/WPD] Remove special type test handling for -flto-visibility-public-std
Needs ReviewPublic

Authored by tejohnson on Jul 14 2020, 8:20 PM.



As discussed in follow up comments on D71913, the special case handling
for -flto-visibility-public-std of not inserting type tests is unneeded,
because that is a Windows option, and the Windows linker does not
support the -lto-whole-program-visibility option which would relax the
visibility. By default, the type tests will get public visibility, which
is what we want for the std classes under that option, and it can't be
relaxed. Therefore the handling that was preventing the type tests from
being inserted (to avoid any visibility relaxation) isn't needed.

Diff Detail