This is an archive of the discontinued LLVM Phabricator instance.

[builtins][Android] Fix 386->686 adjustment
ClosedPublic

Authored by rprichard on Jun 18 2020, 11:57 PM.

Details

Summary

At this point in this code:

  • COMPILER_RT_DEFAULT_TARGET_TRIPLE is "i686-linux-android"
  • arch is "i386"

The get_compiler_rt_target function currently turns that into:

i686-android-linux-android

The ${COMPILER_RT_OS_SUFFIX} is "-android" and redundant, so stop
adding it.

The get_compiler_rt_target() function is used for the
LLVM_ENABLE_PER_TARGET_RUNTIME_DIR mode that isn't normally used with
Android.

Diff Detail

Event Timeline

rprichard created this revision.Jun 18 2020, 11:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 18 2020, 11:57 PM
Herald added subscribers: Restricted Project, mgorny. · View Herald Transcript
phosek accepted this revision.Jun 19 2020, 12:05 AM

LGTM

This revision is now accepted and ready to land.Jun 19 2020, 12:05 AM
This revision was automatically updated to reflect the committed changes.