[PATCH] D26764: [compiler-rt] [cmake] Remove i686 target that is duplicate to i386

Evgenii Stepanov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 29 13:15:17 PDT 2017


eugenis added a comment.

IMHO it was a very bad idea to include the "architecture name" (not even a target triple!) in the library path in the first place. No one else does that. GCC made the right choice and called theirs "libasan.so".

My plan is to tweak the code that sets the library name to use i686 when the target is i386 and the os is android, both in compiler-rt and in clang. There will be no duplicate targets.


Repository:
  rL LLVM

https://reviews.llvm.org/D26764





More information about the cfe-commits mailing list