[PATCH] D37226: [zorg] [sanitizer] Update i686->i386 to match rename in compiler-rt
Evgenii Stepanov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 28 13:03:37 PDT 2017
eugenis added inline comments.
================
Comment at: zorg/buildbot/builders/sanitizers/buildbot_android.sh:68
build_android_ndk arm arm
-build_android_ndk i686 x86
+build_android_ndk i386 x86
----------------
mgorny wrote:
> eugenis wrote:
> > In fact, this might be wrong.
> > Ndk arch should be i686, unlike the standalone toolchain path.
> FWICS ndk arch is actually 'x86' (the second argument). The first seems to be used only to construct the build directory path.
Right. LGTM
Repository:
rL LLVM
https://reviews.llvm.org/D37226
More information about the llvm-commits
mailing list