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

Dan Liew via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 5 15:28:41 PST 2021


delcypher added a subscriber: vitalybuka.
delcypher added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88938



More information about the llvm-commits mailing list