[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