Skip to content

Commit 08c6ad1

Browse files
mizvekovmizvekov
committed
ci: enable ubsan for clang and GCC
1 parent a805e12 commit 08c6ad1

File tree

3 files changed

+45
-61
lines changed

3 files changed

+45
-61
lines changed

‎.github/workflows/ci.yml‎

Lines changed: 18 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,8 @@ jobs:
4141
standards: '20'
4242
latest-factors: |
4343
msvc Optimized-Debug
44-
gcc Coverage
44+
gcc UBSan Coverage
45+
clang UBSan
4546
factors: ''
4647
runs-on: |
4748
apple-clang: macos-15
@@ -64,21 +65,22 @@ jobs:
6465
msvc: ''
6566
extra-values: |
6667
llvm-hash: dd7a3d4d798e30dfe53b5bbbbcd9a23c24ea1af9
67-
llvm-id: {{ substr llvm-hash 0 7 }}
68-
llvm-build-preset-prefix: {{#if optimized-debug}}debwithopt{{else}}{{lowercase build-type}}{{/if}}
68+
llvm-id: {{{ substr llvm-hash 0 7 }}}
69+
llvm-build-preset-prefix: {{#if optimized-debug}}debwithopt{{else}}{{{lowercase build-type}}}{{/if}}
6970
llvm-build-preset-suffix: {{#if (ieq os 'windows') }}win{{else}}unix{{/if}}
70-
llvm-build-preset: {{ llvm-build-preset-prefix }}-{{ llvm-build-preset-suffix }}
71-
llvm-compiler-key: {{ compiler }}-{{ version }}
72-
llvm-cache-key: llvm-libcxx-{{ lowercase os }}-{{ llvm-compiler-key }}-{{ llvm-build-preset-prefix }}-{{ llvm-hash }}
71+
llvm-build-preset: {{{ llvm-build-preset-prefix }}}-{{{ llvm-build-preset-suffix }}}
72+
llvm-compiler-key: {{{ compiler }}}-{{{ version }}}
73+
llvm-cache-key: llvm-libcxx-{{{ lowercase os }}}-{{{ llvm-compiler-key }}}-{{{ llvm-build-preset-prefix }}}-{{{ ubsan }}}-{{{ asan }}}-{{{ msan }}}-{{{ llvm-hash }}}
7374
llvm-root: ../third-party/llvm-project/install
74-
llvm-archive-basename: llvm-{{ os }}-{{ llvm-build-preset-prefix }}-{{ llvm-id }}
75+
llvm-archive-basename: llvm-{{{ os }}}-{{{ llvm-build-preset-prefix }}}-{{{ llvm-id }}}
7576
llvm-archive-extension: {{#if (ieq os 'windows') }}7z{{else}}tar.bz2{{/if}}
76-
llvm-archive-filename: {{ llvm-archive-basename }}.{{ llvm-archive-extension }}
77-
mrdocs-ccflags: {{ ccflags }} {{#if (eq compiler 'gcc') }}-static{{/if}} {{#if asan }}-static-libasan{{/if}} {{#if tsan }}-static-libtsan{{/if}}
78-
mrdocs-cxxflags: {{ cxxflags }} {{#if (eq compiler 'gcc') }}-static{{/if}} {{#if asan }}-static-libasan{{/if}} {{#if tsan }}-static-libtsan{{/if}}
77+
llvm-archive-filename: {{{ llvm-archive-basename }}}.{{{ llvm-archive-extension }}}
78+
mrdocs-ccflags: {{{ ccflags }}} {{#if (and (eq compiler 'gcc') (not asan)) }}-static{{/if}}
79+
mrdocs-cxxflags: {{{ cxxflags }}} {{#if (and (eq compiler 'gcc') (not asan)) }}-static{{/if}}
7980
mrdocs-package-generators: {{#if (ieq os 'windows') }}7Z ZIP WIX{{else}}TGZ TXZ{{/if}}
80-
mrdocs-release-package-artifact: release-packages-{{ lowercase os }}
81+
mrdocs-release-package-artifact: release-packages-{{{ lowercase os }}}
8182
output-file: matrix.json
83+
trace-commands: true
8284

8385
# Set up the version as expected by the LLVM matrix script and @actions/core
8486
- name: Setup Node.js
@@ -214,7 +216,7 @@ jobs:
214216

215217
- name: Download LLVM Binaries
216218
id: llvm-download
217-
if: steps.llvm-cache.outputs.cache-hit != 'true'
219+
if: steps.llvm-cache.outputs.cache-hit != 'true' && !(matrix.ubsan || matrix.asan || matrix.msan)
218220
run: |
219221
set -x
220222
url=https://mrdocs.com/llvm+clang/${{ matrix.llvm-archive-filename }}
@@ -259,6 +261,10 @@ jobs:
259261
build-type: ${{ matrix.build-type }}
260262
extra-args: |
261263
${{ runner.os == 'Windows' && '-DLLVM_ENABLE_RUNTIMES=libcxx' || '-DLLVM_ENABLE_RUNTIMES=libcxx;libcxxabi;libunwind' }}
264+
# The UBSan vptr sanitizer needs RTTI.
265+
${{ matrix.ubsan && '-DLLVM_ENABLE_RTTI=ON' }}
266+
# The LLVM_USE_SANITIZER option doesn't support GCC.
267+
${{ !matrix.compiler == 'gcc' && matrix.ubsan && '-DLLVM_USE_SANITIZER=Undefined' }}
262268
cc: ${{ steps.setup-cpp.outputs.cc }}
263269
cxx: ${{ steps.setup-cpp.outputs.cxx }}
264270
generator: Ninja

‎src/lib/Support/Yaml.cpp‎

Lines changed: 25 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -15,51 +15,33 @@
1515
namespace clang {
1616
namespace mrdocs {
1717

18-
void
19-
YamlReporter::
20-
diag(
21-
llvm::SMDiagnostic const& D)
22-
{
23-
report::Level level = report::Level::debug;
24-
switch(D.getKind())
25-
{
26-
case llvm::SourceMgr::DiagKind::DK_Remark:
27-
level = report::Level::debug;
28-
break;
29-
case llvm::SourceMgr::DiagKind::DK_Note:
30-
level = report::Level::info;
31-
break;
32-
case llvm::SourceMgr::DiagKind::DK_Warning:
33-
{
34-
if(D.getMessage().starts_with("unknown key "))
35-
{
36-
// don't show these
37-
return;
38-
}
39-
level = report::Level::warn;
40-
break;
18+
void YamlReporter::diag(llvm::SMDiagnostic const &D, void *) {
19+
report::Level level = report::Level::debug;
20+
switch (D.getKind()) {
21+
case llvm::SourceMgr::DiagKind::DK_Remark:
22+
level = report::Level::debug;
23+
break;
24+
case llvm::SourceMgr::DiagKind::DK_Note:
25+
level = report::Level::info;
26+
break;
27+
case llvm::SourceMgr::DiagKind::DK_Warning: {
28+
if (D.getMessage().starts_with("unknown key ")) {
29+
// don't show these
30+
return;
4131
}
42-
case llvm::SourceMgr::DiagKind::DK_Error:
43-
level = report::Level::error;
44-
break;
45-
default:
46-
MRDOCS_UNREACHABLE();
47-
}
48-
49-
report::call_impl(
50-
level,
51-
[&](llvm::raw_ostream& os)
52-
{
53-
D.print("mrdocs", os, true, true);
54-
}, nullptr);
55-
}
32+
level = report::Level::warn;
33+
break;
34+
}
35+
case llvm::SourceMgr::DiagKind::DK_Error:
36+
level = report::Level::error;
37+
break;
38+
default:
39+
MRDOCS_UNREACHABLE();
40+
}
5641

57-
void
58-
YamlReporter::
59-
diagFnImpl(
60-
llvm::SMDiagnostic const& D, void* ctx)
61-
{
62-
reinterpret_cast<YamlReporter*>(ctx)->diag(D);
42+
report::call_impl(
43+
level, [&](llvm::raw_ostream &os) { D.print("mrdocs", os, true, true); },
44+
nullptr);
6345
}
6446

6547
} // mrdocs

‎src/lib/Support/Yaml.hpp‎

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,16 +21,12 @@ namespace mrdocs {
2121
class MRDOCS_DECL
2222
YamlReporter
2323
{
24-
void diag(llvm::SMDiagnostic const&);
25-
static void diagFnImpl(llvm::SMDiagnostic const&, void*);
24+
static void diag(llvm::SMDiagnostic const &, void *);
2625

2726
public:
2827
typedef void (*diagFn)(llvm::SMDiagnostic const&, void*);
2928

30-
operator diagFn() const noexcept
31-
{
32-
return &diagFnImpl;
33-
}
29+
operator diagFn() const noexcept { return &diag; }
3430
};
3531

3632
} // mrdocs

0 commit comments

Comments
 (0)