[PATCH] D69196: Fix lld detection in standalone compiler-rt.

[Evgenii Stepanov via Phabricator via llvm-commits]

eugenis marked an inline comment as done.
eugenis added a comment.

As it turns out, hwasan has not been tested on Android for the past two months.
It was broken by this change (which went in without code review, btw, not cool):

  http://llvm.org/viewvc/llvm-project?rev=368242&view=rev

The change may look trivial, but nothing in CMake ever is.

Now, this can not go in right now, because, naturally, most of the stuff we have committed over the past two months does not pass on Android.
It looks like it is almost all Peter.
One thing that I've noticed is that __hwasan_personality_wrapper has neither extern"C" or public visibility, so it is not exported from the shared runtime library.



================
Comment at: compiler-rt/CMakeLists.txt:500
   else()
-    set(COMPILER_RT_HAS_LLD FALSE)
+    set(COMPILER_RT_HAS_LLD COMPILER_RT_HAS_FUSE_LD_LLD_FLAG)
   endif()
----------------
I believe all the stuff above this line is necessary for non-standalone case, when the linker flag check would run before the linker itself is built.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69196/new/

https://reviews.llvm.org/D69196