This is an archive of the discontinued LLVM Phabricator instance.

gn build: Fix redundant object files in builtin lib.
ClosedPublic

Authored by thakis on Jul 31 2019, 6:54 AM.

Details

Summary

compiler-rt's builtin library has generic implementations of many
functions, and then per-arch optimized implementations of some.

In the CMake build, both filter_builtin_sources() and an explicit loop
at the end of the build file (see D37166) filter out the generic
versions if a per-arch file is present.

The GN build wasn't doing this filtering. Just do the filtering manually
and explicitly, instead of being clever.

While here, also remove files from the mingw/arm build that are
redundantly listed after D39938 / r318139 (both from the CMake and the
GN build).

While here, also fix a target_os -> target_cpu typo.

Diff Detail

Repository
rL LLVM

Event Timeline

thakis created this revision.Jul 31 2019, 6:54 AM
thakis updated this revision to Diff 212590.Jul 31 2019, 8:35 AM
thakis edited the summary of this revision. (Show Details)

target_os == "x86" fix

phosek accepted this revision.Jul 31 2019, 9:07 AM

LGTM

This revision is now accepted and ready to land.Jul 31 2019, 9:07 AM
This revision was automatically updated to reflect the committed changes.

Thanks for cleaning up the stuff I left behind!