[PATCH] D41809: Clang counterpart change for fuzzer FreeBSD support
krytarowski added a comment. Committed as SVN r. 322469. Repository: rL LLVM https://reviews.llvm.org/D41809 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D41809: Clang counterpart change for fuzzer FreeBSD support
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL322469: Clang counterpart change for fuzzer FreeBSD support (authored by kamil, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D41809?vs=129364=129797#toc Repository: rL LLVM https://reviews.llvm.org/D41809 Files: cfe/trunk/lib/Driver/ToolChains/FreeBSD.cpp Index: cfe/trunk/lib/Driver/ToolChains/FreeBSD.cpp === --- cfe/trunk/lib/Driver/ToolChains/FreeBSD.cpp +++ cfe/trunk/lib/Driver/ToolChains/FreeBSD.cpp @@ -392,6 +392,8 @@ } if (IsX86 || IsX86_64) { Res |= SanitizerKind::SafeStack; +Res |= SanitizerKind::Fuzzer; +Res |= SanitizerKind::FuzzerNoLink; } return Res; } Index: cfe/trunk/lib/Driver/ToolChains/FreeBSD.cpp === --- cfe/trunk/lib/Driver/ToolChains/FreeBSD.cpp +++ cfe/trunk/lib/Driver/ToolChains/FreeBSD.cpp @@ -392,6 +392,8 @@ } if (IsX86 || IsX86_64) { Res |= SanitizerKind::SafeStack; +Res |= SanitizerKind::Fuzzer; +Res |= SanitizerKind::FuzzerNoLink; } return Res; } ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D41809: Clang counterpart change for fuzzer FreeBSD support
martell accepted this revision. martell added a comment. Given that https://reviews.llvm.org/rL322380 has been merged this seems good to me. https://reviews.llvm.org/D41809 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D41809: Clang counterpart change for fuzzer FreeBSD support
krytarowski added a comment. Looks good, please improve the description of the commit (and not your authorship that will be preserved in the commit message). Once accepted by another developer, I can commit it for you. https://reviews.llvm.org/D41809 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D41809: Clang counterpart change for fuzzer FreeBSD support
devnexen added a comment. In https://reviews.llvm.org/D41809#969475, @kimgr wrote: > Typo in the commit title: buzzer :) Changed ... just to follow-up the now accepted change of this one https://reviews.llvm.org/D41642 https://reviews.llvm.org/D41809 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D41809: Clang counterpart change for fuzzer FreeBSD support
devnexen updated this revision to Diff 129364. https://reviews.llvm.org/D41809 Files: lib/Driver/ToolChains/FreeBSD.cpp Index: lib/Driver/ToolChains/FreeBSD.cpp === --- lib/Driver/ToolChains/FreeBSD.cpp +++ lib/Driver/ToolChains/FreeBSD.cpp @@ -392,6 +392,8 @@ } if (IsX86 || IsX86_64) { Res |= SanitizerKind::SafeStack; +Res |= SanitizerKind::Fuzzer; +Res |= SanitizerKind::FuzzerNoLink; } return Res; } Index: lib/Driver/ToolChains/FreeBSD.cpp === --- lib/Driver/ToolChains/FreeBSD.cpp +++ lib/Driver/ToolChains/FreeBSD.cpp @@ -392,6 +392,8 @@ } if (IsX86 || IsX86_64) { Res |= SanitizerKind::SafeStack; +Res |= SanitizerKind::Fuzzer; +Res |= SanitizerKind::FuzzerNoLink; } return Res; } ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits