https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/141814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/141814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,74 @@
+// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
UTC_ARGS: --version 5
+// RUN: %clang_cc1 -emit-llvm -triple x86_64 -std=c17 -fsanitize=function %s
-o - \
vitalybuka wrote:
-O1 or -O2 to reduce the bloat?
https:/
https://github.com/vitalybuka closed
https://github.com/llvm/llvm-project/pull/139772
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka reopened
https://github.com/llvm/llvm-project/pull/139772
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -44,13 +44,11 @@ bool NoSanitizeList::containsFunction(SanitizerMask Mask,
bool NoSanitizeList::containsFile(SanitizerMask Mask, StringRef FileName,
StringRef Category) const {
- unsigned NoSanLine = SSCL->inSectionBlame(Mask, "src", FileN
@@ -70,7 +70,7 @@ class FileSystem;
/// ---
class SpecialCaseList {
public:
- constexpr static std::pair NotFound = {0, 0};
+ static constexpr std::pair NotFound = {0, 0};
vitalybuka wrote:
yes, for consistency
https://github.com/llvm/llvm-project/pull/1415
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/141540
>From e7540c31c69df827d646e88b271a355c587bd9da Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Tue, 27 May 2025 02:36:29 +
Subject: [PATCH 1/5] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20ch?=
=?U
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/141540
>From e7540c31c69df827d646e88b271a355c587bd9da Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Tue, 27 May 2025 02:36:29 +
Subject: [PATCH 1/4] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20ch?=
=?U
https://github.com/vitalybuka approved this pull request.
Please double check and merge
https://github.com/llvm/llvm-project/pull/141540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/141540
>From e7540c31c69df827d646e88b271a355c587bd9da Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Tue, 27 May 2025 02:36:29 +
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20ch?=
=?U
vitalybuka wrote:
I have local changes, I'll push update
https://github.com/llvm/llvm-project/pull/141540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 01/17] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
=?U
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 01/16] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
=?U
@@ -56,10 +56,20 @@ void SanitizerSpecialCaseList::createSanitizerSections() {
bool SanitizerSpecialCaseList::inSection(SanitizerMask Mask, StringRef Prefix,
StringRef Query,
StringRef Category) c
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/140964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -125,7 +125,7 @@ class SpecialCaseList {
// Returns zero if no match is found.
LLVM_ABI unsigned match(StringRef Query) const;
-StringMap> Globs;
+std::vector>>
Globs;
vitalybuka wrote:
Glob(Glob&&) = delete; // no copy, no move, because
@@ -125,7 +125,7 @@ class SpecialCaseList {
// Returns zero if no match is found.
LLVM_ABI unsigned match(StringRef Query) const;
-StringMap> Globs;
+std::vector>>
Globs;
vitalybuka wrote:
Could be easier with:
```
class Glob {
name
patt
@@ -53,24 +53,27 @@ Error SpecialCaseList::Matcher::insert(StringRef Pattern,
unsigned LineNumber,
return Error::success();
}
- auto [It, DidEmplace] = Globs.try_emplace(Pattern);
- if (DidEmplace) {
-// We must be sure to use the string in the map rather than the
@@ -553,7 +553,10 @@ void
WarningsSpecialCaseList::processSections(DiagnosticsEngine &Diags) {
// Each section has a matcher with that section's name, attached to that
// line.
const auto &DiagSectionMatcher = Entry.SectionMatcher;
-unsigned DiagLine = DiagSect
@@ -71,9 +76,9 @@ unsigned SpecialCaseList::Matcher::match(StringRef Query)
const {
for (const auto &[Pattern, Pair] : Globs)
if (Pair.first.match(Query))
return Pair.second;
- for (const auto &[Regex, LineNumber] : RegExes)
-if (Regex->match(Query))
- re
@@ -63,6 +63,11 @@ Error SpecialCaseList::Matcher::insert(StringRef Pattern,
unsigned LineNumber,
.moveInto(Pair.first))
return Err;
Pair.second = LineNumber;
+ } else {
vitalybuka wrote:
Please move llvm/lib/Support/SpecialC
@@ -103,6 +101,22 @@ supported sanitizers.
char c = toobig; // also not instrumented
}
+Conflicting entries are resolved by the latest entry, which takes precedence.
vitalybuka wrote:
If multiple entries match the source, than the latest entry takes the
@@ -63,6 +63,11 @@ Error SpecialCaseList::Matcher::insert(StringRef Pattern,
unsigned LineNumber,
.moveInto(Pair.first))
return Err;
Pair.second = LineNumber;
+ } else {
vitalybuka wrote:
And "Pair.second = LineNumber;" is no
https://github.com/vitalybuka created
https://github.com/llvm/llvm-project/pull/140855
None
>From 46288061d9d6acbed8cb3ada1fb2265c379f5d67 Mon Sep 17 00:00:00 2001
From: Vitaly Buka
Date: Tue, 20 May 2025 23:45:52 -0700
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -63,6 +63,11 @@ Error SpecialCaseList::Matcher::insert(StringRef Pattern,
unsigned LineNumber,
.moveInto(Pair.first))
return Err;
Pair.second = LineNumber;
+ } else {
vitalybuka wrote:
Also we need a patch to Matcher::Glob
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -77,10 +77,8 @@ For example, supplying the above ``ignorelist.txt`` to
``-fsanitize-ignorelist=ignorelist.txt`` disables overflow sanitizer
instrumentation for arithmetic operations containing values of type ``int``.
-The ``=sanitize`` category is also supported. Any types
@@ -58,7 +58,7 @@ Usage with UndefinedBehaviorSanitizer
ability to adjust instrumentation based on type.
By default, supported sanitizers will have their instrumentation disabled for
-types specified within an ignorelist.
+entris specified within an ignorelist.
---
@@ -960,6 +960,7 @@ Sanitizers
--
- ``-fsanitize=vptr`` is no longer a part of ``-fsanitize=undefined``.
+- UBSan ignorelists now support the syntax ``src:*=sanitize``.
vitalybuka wrote:
I think it applies to all Sanitizers, with src: syntax
https://
@@ -78,9 +78,7 @@ For example, supplying the above ``ignorelist.txt`` to
instrumentation for arithmetic operations containing values of type ``int``.
The ``=sanitize`` category is also supported. Any types assigned to the
vitalybuka wrote:
type -> entry?
htt
vitalybuka wrote:
Also I don't mind we fix multifile and sections in followup patches.
But this one needs at least documentation update.
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
vitalybuka wrote:
Note: Use to download my updates `spr patch 140529`
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3,11 +3,11 @@
// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test2.c -o -
| FileCheck %s --check-prefixes=CHECK2
-// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=sig
@@ -3,11 +3,11 @@
// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test2.c -o -
| FileCheck %s --check-prefixes=CHECK2
-// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=sig
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?U
Author: Vitaly Buka
Date: 2025-05-19T15:45:43-07:00
New Revision: 5db4aeae3c413d586faed03771626818d6a807a9
URL:
https://github.com/llvm/llvm-project/commit/5db4aeae3c413d586faed03771626818d6a807a9
DIFF:
https://github.com/llvm/llvm-project/commit/5db4aeae3c413d586faed03771626818d6a807a9.diff
L
Author: Vitaly Buka
Date: 2025-05-19T15:42:59-07:00
New Revision: b93bc773f86b3746f8023c5dda9a8ab892fa0499
URL:
https://github.com/llvm/llvm-project/commit/b93bc773f86b3746f8023c5dda9a8ab892fa0499
DIFF:
https://github.com/llvm/llvm-project/commit/b93bc773f86b3746f8023c5dda9a8ab892fa0499.diff
L
vitalybuka wrote:
llvm-project/clang/docs/SanitizerSpecialCaseList.rst needs to be updated
And nice to mention in llvm-project/clang/docs/ReleaseNotes.rst
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,17 +1,16 @@
// RUN: rm -rf %t
// RUN: split-file %s %t
-// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test1.c -o -
| FileCheck %s --check-prefixes=CHECK1,IGNORE
+// RUN: %c
@@ -44,6 +44,12 @@ bool NoSanitizeList::containsFunction(SanitizerMask Mask,
bool NoSanitizeList::containsFile(SanitizerMask Mask, StringRef FileName,
StringRef Category) const {
+ unsigned NoSanLine = SSCL->inSectionBlame(Mask, "src", FileNa
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?U
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka closed
https://github.com/llvm/llvm-project/pull/140602
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?U
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140602
>From 210f005ecccbbd4fed439d3e156d5e0146807bed Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 19:49:33 +
Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?U
@@ -0,0 +1,37 @@
+// RUN: rm -rf %t
vitalybuka wrote:
But in this case we failed to do so, if reordering of patches is needed, SPR
often fails.
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140602
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/140602
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140602
>From 210f005ecccbbd4fed439d3e156d5e0146807bed Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 19:49:33 +
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
=?U
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?U
@@ -0,0 +1,37 @@
+// RUN: rm -rf %t
vitalybuka wrote:
THe point of SPR to have stacked PR, so this one will already be rebased on
#140602
https://github.com/llvm/llvm-project/pull/140529
___
cfe-commits mailing list
vitalybuka wrote:
> > The linked pull request (#140529) introduces a new feature rather than
> > correcting a bug.
>
> I would argue that since the `src:*=sanitize` syntax could reasonably be
> expected to work, but it did not work, then that is both a missing feature
> and a bug (not working
@@ -0,0 +1,43 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test1.c -o -
| FileCheck %s --check-prefixes=CHECK1,IGNORE
+// RUN: %cl
@@ -0,0 +1,43 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test1.c -o -
| FileCheck %s --check-prefixes=CHECK1,IGNORE
@@ -0,0 +1,37 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test1.c -o - | FileCheck
%s -check-prefix=CHECK-ALLOWLIST
+// RUN: %clang_cc1 -trip
@@ -0,0 +1,37 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test1.c -o - | FileCheck
%s -check-prefix=CHECK-ALLOWLIST
vitalybu
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/140602
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/140602
>From 210f005ecccbbd4fed439d3e156d5e0146807bed Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 19:49:33 +
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
=?U
https://github.com/vitalybuka approved this pull request.
I don't have issues with reformatting the entire files, and
https://www.llvm.org/docs/Contributing.html says nothing about that.
But I guess convention is to reformat only parts you are going to change - less
noise in blame output, and
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/140309
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2779,6 +2779,29 @@ void CodeGenFunction::EmitTypeMetadataCodeForVCall(const
CXXRecordDecl *RD,
}
}
+/// Converts the CFITypeCheckKind into SanitizerKind::SanitizerOrdinal and
+/// llvm::SanitizerStatKind.
+static std::pair
+SanitizerInfoFromCFICheckKind(CodeGenFunction:
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/139809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vitalybuka wrote:
LGTM
https://github.com/llvm/llvm-project/pull/140117
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka approved this pull request.
Please use tag [NFC] or [NFCI] if the patch does not change any functionality.
https://github.com/llvm/llvm-project/pull/140127
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
@@ -132,14 +132,15 @@ class SpecialCaseList {
using SectionEntries = StringMap>;
struct Section {
-Section(std::unique_ptr M) : SectionMatcher(std::move(M)){};
-Section() : Section(std::make_unique()) {}
+Section(std::unique_ptr M) : SectionMatcher(std::move(M)
@@ -38,19 +38,18 @@ SanitizerSpecialCaseList::createOrDie(const
std::vector &Paths,
void SanitizerSpecialCaseList::createSanitizerSections() {
for (auto &It : Sections) {
vitalybuka wrote:
`auto& S : `
https://github.com/llvm/llvm-project/pull/140127
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/139965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka deleted
https://github.com/llvm/llvm-project/pull/139809
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3353,6 +3353,17 @@ class CodeGenFunction : public CodeGenTypeCache {
SanitizerSet SkippedChecks = SanitizerSet(),
llvm::Value *ArraySize = nullptr);
+ /// Returns debug info, with additional annotation if enabled by
+ /// CGM.get
@@ -3353,6 +3353,16 @@ class CodeGenFunction : public CodeGenTypeCache {
SanitizerSet SkippedChecks = SanitizerSet(),
llvm::Value *ArraySize = nullptr);
+ /// Returns debug info, with additional annotation if enabled by
+ /// CGM.get
@@ -2842,24 +2882,12 @@ void CodeGenFunction::EmitVTablePtrCheck(const
CXXRecordDecl *RD,
!CGM.HasHiddenLTOVisibility(RD))
return;
- SanitizerKind::SanitizerOrdinal M;
- llvm::SanitizerStatKind SSK;
+ auto [M, SSK] = ParseCFITypeCheckKind(TCK);
switch (TCK) {
@@ -2779,13 +2779,50 @@ void
CodeGenFunction::EmitTypeMetadataCodeForVCall(const CXXRecordDecl *RD,
}
}
+std::pair
+CodeGenFunction::ParseCFITypeCheckKind(CFITypeCheckKind TCK) {
+ SanitizerKind::SanitizerOrdinal M;
+ llvm::SanitizerStatKind SSK;
+
+ switch (TCK) {
+ ca
@@ -1217,6 +1217,35 @@ void CodeGenFunction::EmitBoundsCheck(const Expr *E,
const Expr *Base,
EmitBoundsCheckImpl(E, Bound, Index, IndexType, IndexedType, Accessed);
}
+llvm::DILocation *CodeGenFunction::SanitizerAnnotateDebugInfo(
vitalybuka wrote:
This g
@@ -0,0 +1,23 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test1.c -o - | FileCheck
%s -check-prefix=CHECK-ALLOWLIST
+// RUN: %clang_cc1 -trip
@@ -44,7 +44,8 @@ bool NoSanitizeList::containsFunction(SanitizerMask Mask,
bool NoSanitizeList::containsFile(SanitizerMask Mask, StringRef FileName,
StringRef Category) const {
- return SSCL->inSection(Mask, "src", FileName, Category);
+ bo
@@ -0,0 +1,23 @@
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+// RUN: %clang_cc1 -triple x86_64-linux-gnu -fsanitize=signed-integer-overflow
-fsanitize-ignorelist=%t/src.ignorelist -emit-llvm %t/test1.c -o - | FileCheck
%s -check-prefix=CHECK-ALLOWLIST
+// RUN: %clang_cc1 -trip
vitalybuka wrote:
Reminder to @thurstond as well :)
Please figure out https://llvm.org/docs/GitHub.html#stacked-pull-requests
https://github.com/llvm/llvm-project/pull/139772
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
@@ -44,7 +44,8 @@ bool NoSanitizeList::containsFunction(SanitizerMask Mask,
bool NoSanitizeList::containsFile(SanitizerMask Mask, StringRef FileName,
StringRef Category) const {
- return SSCL->inSection(Mask, "src", FileName, Category);
+ bo
vitalybuka wrote:
> I am not a fan of the change to the function used for array-bounds.
More specific? I like it in general.
It's about just about the name of a new fake function?
https://github.com/llvm/llvm-project/pull/139809
___
cfe-commits mailin
vitalybuka wrote:
> sanitizer-aarch64-linux-bootstrap-asan
Sorry, this one caused by my zorg changes.
https://github.com/llvm/llvm-project/pull/139398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/139335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka closed
https://github.com/llvm/llvm-project/pull/139335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vitalybuka wrote:
I checked on the bot, this fixes the leak!
Thank you!
https://github.com/llvm/llvm-project/pull/139335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vitalybuka wrote:
```
=
==3963603==ERROR: LeakSanitizer: detected memory leaks
Direct leak of 576 byte(s) in 1 object(s) allocated from:
#0 0xb3001e6d179c in operator new(unsigned long)
/home/b/sanitizer-aarch64-linux-bootstrap
vitalybuka wrote:
> That passes for me. I can't even figure out which unit test is failing, they
> all say `OK` and not `FAILED`. The one error in there looks like it's
> expected since its test still returns `OK`. Are we totally sure this isn't an
> infrastructure issue?
Unlikely.
Process ex
vitalybuka wrote:
This one is still broken
https://lab.llvm.org/buildbot/#/builders/55/builds/11054/steps/11/logs/stdio
https://github.com/llvm/llvm-project/pull/138234
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
@@ -0,0 +1,47 @@
+// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
UTC_ARGS: --version 5
+// RUN: %clang_cc1 -triple x86_64-unknown-linux -O0 -fsanitize-cfi-cross-dso \
vitalybuka wrote:
would we have less boilerplate if we do e.g. "
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/138323
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1078,22 +1091,61 @@ void
ModuleSanitizerCoverage::InjectCoverageAtBlock(Function &F, BasicBlock &BB,
Store->setNoSanitizeMetadata();
}
if (Options.StackDepth && IsEntryBB && !IsLeafFunc) {
-// Check stack depth. If it's the deepest so far, record it.
Modu
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/138577
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2533,6 +2533,31 @@ def fno_sanitize_merge_handlers : Flag<["-"],
"fno-sanitize-merge">, Group,
AliasArgs<["all"]>,
Visibility<[ClangOption, CLOption]>,
HelpText<"Do not allow compiler to merge handlers for
any sanitizers">;
vitalybuka wrote:
In description ?
> Florian1
@fmayer would be nicer for cross-linking
https://github.com/llvm/llvm-project/pull/138577
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/138577
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 1406 matches
Mail list logo