[PATCH] D96337: [sanitizer] Fix suffix-log-path_test.c on arm-linux-gnu

Adhemerval Zanella via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 11 04:12:27 PST 2021


zatrazz added a comment.

In D96337#2556270 <https://reviews.llvm.org/D96337#2556270>, @ostannard wrote:

> Could this also affect regular users of the sanitizers, or is it specific to the way we are running the tests? If it could affect users, then could we fix this in the clang driver, so that unwind tables are turned on by default whenever the sanitizers need them?

I think it might affect regular uses on arm-linux-gnueabi, specially if they mix thumb generated object built gcc (which uses a slight different scheme to layout the FP than clang). However, I think it is a orthogonal issue, since compiler-rt can be built as a standalone library. I will work on adding this support on clang driver.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96337



More information about the llvm-commits mailing list