[PATCH] D98286: [Driver] Enable kernel address and memory sanitizers on FreeBSD

2021-04-15 Thread Alexander Richardson via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG99eca1bd9c7a: [Driver] Enable kernel address and memory sanitizers on FreeBSD (authored by markj, committed by arichardson). Repository: rG LLVM G

[PATCH] D98286: [Driver] Enable kernel address and memory sanitizers on FreeBSD

2021-03-24 Thread Mark Johnston via Phabricator via cfe-commits
markj updated this revision to Diff 333106. markj added a comment. - Add a couple of regression tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98286/new/ https://reviews.llvm.org/D98286 Files: clang/lib/Driver/ToolChains/FreeBSD.cpp clan

[PATCH] D98286: [Driver] Enable kernel address and memory sanitizers on FreeBSD

2021-03-24 Thread Mark Johnston via Phabricator via cfe-commits
markj added a comment. In D98286#2648449 , @dim wrote: > LGTM, but this would probably need some sort of test, at least that it > correctly accepts the flag(s) now? Working on it, I see some examples. Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D98286: [Driver] Enable kernel address and memory sanitizers on FreeBSD

2021-03-24 Thread Dimitry Andric via Phabricator via cfe-commits
dim accepted this revision. dim added a comment. LGTM, but this would probably need some sort of test, at least that it correctly accepts the flag(s) now? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98286/new/ https://reviews.llvm.org/D98286 __

[PATCH] D98286: [Driver] Enable kernel address and memory sanitizers on FreeBSD

2021-03-24 Thread Mark Johnston via Phabricator via cfe-commits
markj added a comment. In D98286#2647606 , @arichardson wrote: > Does it also work on AArch64? Not yet, I haven't yet ported the sanitizer runtimes (though it's on my todo list). ASAN requires that we specify the relative offset of the shadow map to the

[PATCH] D98286: [Driver] Enable kernel address and memory sanitizers on FreeBSD

2021-03-24 Thread Alexander Richardson via Phabricator via cfe-commits
arichardson accepted this revision. arichardson added a comment. This revision is now accepted and ready to land. Does it also work on AArch64? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98286/new/ https://reviews.llvm.org/D98286 __

[PATCH] D98286: [Driver] Enable kernel address and memory sanitizers on FreeBSD

2021-03-24 Thread Mark Johnston via Phabricator via cfe-commits
markj updated this revision to Diff 332974. markj added a comment. Enable the memory sanitizer too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98286/new/ https://reviews.llvm.org/D98286 Files: clang/lib/Driver/ToolChains/FreeBSD.cpp Index: