This is an archive of the discontinued LLVM Phabricator instance.

gn build: Add support for cross-compiling the builtins and profile runtimes for Android aarch64 and arm.
ClosedPublic

Authored by pcc on Oct 31 2019, 1:38 PM.

Diff Detail

Event Timeline

pcc created this revision.Oct 31 2019, 1:38 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 31 2019, 1:38 PM
phosek accepted this revision.Oct 31 2019, 2:58 PM

LGTM

llvm/utils/gn/secondary/compiler-rt/lib/builtins/BUILD.gn
372

Is current_cpu_is_vfp used anywhere else? If not I'd just inline it.

This revision is now accepted and ready to land.Oct 31 2019, 2:58 PM
pcc marked an inline comment as done.Oct 31 2019, 3:48 PM
pcc added inline comments.
llvm/utils/gn/secondary/compiler-rt/lib/builtins/BUILD.gn
372

Yeah, I wasn't sure whether inlining it would be clear enough but maybe the vfp substring in the names of all the files below is enough.

This revision was automatically updated to reflect the committed changes.