[PATCH] D88938: [llvm-xsan] Introduce llvm-xsan

Fri Mar 5 16:24:03 PST 2021

vitalybuka added a comment.

In D88938#2608062 <https://reviews.llvm.org/D88938#2608062>, @delcypher wrote:

> In D88938#2607598 <https://reviews.llvm.org/D88938#2607598>, @vitalybuka wrote:
>
>> I can't see "rdar" content and motivations for these scripts, and I don't expect myself extend my workflow with another layer between me and sanitizers.
>> As a negative side, if it's committed it will be community responsibility to maintain. 
>> But I have no strong opinion so I am going to resign.
>
> @vitalybuka Fair enough. Sorry I think you had a question here that's now deleted that I missed. I think it was a question about why this is in `compiler-rt` and not `llvm`. The reason for that is because many of the tests will require parsing the output of the Sanitizers which is defined in the runtime. The most convenient place to run binaries built with Sanitizers to parse their output is compiler-rt  because we already do that extensively there to test the Sanitizers.

Actually question was "Why this is a part of llvm and not compiler-rt?". But it's already in compiler-rt, so I've deleted the question :)

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88938