================
@@ -56,10 +56,20 @@ void SanitizerSpecialCaseList::createSanitizerSections() {
bool SanitizerSpecialCaseList::inSection(SanitizerMask Mask, StringRef Prefix,
StringRef Query,
StringRef Category) const {
- for (auto &S : SanitizerSections)
- if ((S.Mask & Mask) &&
- SpecialCaseList::inSectionBlame(S.Entries, Prefix, Query, Category))
- return true;
+ return inSectionBlame(Mask, Prefix, Query, Category);
+}
- return false;
+unsigned SanitizerSpecialCaseList::inSectionBlame(SanitizerMask Mask,
+ StringRef Prefix,
+ StringRef Query,
+ StringRef Category) const {
+ for (auto &S : SanitizerSections) {
----------------
qinkunbao wrote:
Good catch. Thanks for the finding.
New tests:
[6be9bfd](https://github.com/llvm/llvm-project/pull/140529/commits/6be9bfd1a21c1ef936279b4e207a9e49300bb16c)
Fix:
[f582891](https://github.com/llvm/llvm-project/pull/140529/commits/f582891bd77b4ad1f9829350bbb96153831dc89a)
The Windows CI check should be irrelavant to this PR
https://github.com/llvm/llvm-project/commit/de93f7ed0d615060735ad15e720f2497ed1d2468
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
