[PATCH] D43148: Adding msan support for FreeBSD

2018-02-15 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC325241: Adding msan support for FreeBSD (authored by kamil, committed by ). Repository: rC Clang https://reviews.llvm.org/D43148 Files: lib/Driver/ToolChains/FreeBSD.cpp Index:

[PATCH] D43148: Adding msan support for FreeBSD

2018-02-14 Thread David CARLIER via Phabricator via cfe-commits
devnexen updated this revision to Diff 134331. https://reviews.llvm.org/D43148 Files: lib/Driver/ToolChains/FreeBSD.cpp Index: lib/Driver/ToolChains/FreeBSD.cpp === --- lib/Driver/ToolChains/FreeBSD.cpp +++

[PATCH] D43148: Adding msan support for FreeBSD

2018-02-14 Thread Kamil Rytarowski via Phabricator via cfe-commits
krytarowski added inline comments. Comment at: FreeBSD.cpp:397 Res |= SanitizerKind::FuzzerNoLink; +if (IsX86_64) +Res |= SanitizerKind::Memory; I would keep it in a dedicated `if (IsX86_64)` (similar to NetBSD). Otherwise it looks fine.

[PATCH] D43148: Adding msan support for FreeBSD

2018-02-09 Thread David CARLIER via Phabricator via cfe-commits
devnexen created this revision. devnexen added a reviewer: krytarowski. devnexen created this object with visibility "All Users". Herald added subscribers: cfe-commits, emaste. Enabling memory sanitiser for X86_64 arch only. To match the sanitiser counterpart. Repository: rC Clang