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

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 29 15:29:59 PDT 2017


rnk added a comment.

In https://reviews.llvm.org/D26764#855791, @eugenis wrote:

> 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".


+1, one day we should untangle the compiler-rt library names.

> 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