Skip to content

Commit 6421727

Browse files
Fix test failure
Reopen #114270
1 parent b7eb40d commit 6421727

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

clang/test/Analysis/taint-generic.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,11 @@ void top() {
158158
clang_analyzer_isTainted(E); // expected-warning {{NO}}
159159

160160
Aggr A = mySource1<Aggr>();
161-
clang_analyzer_isTainted(A); // expected-warning {{YES}}
161+
// FIXME Ideally, both A and A.data should be tainted. However, the
162+
// implementation used by e5ac9145ba29 ([analyzer][taint] Recognize
163+
// tainted LazyCompoundVals (4/4) (#115919), 2024-11-15) led to FPs and
164+
// FNs in various scenarios and had to be reverted to fix #153782.
165+
clang_analyzer_isTainted(A); // expected-warning {{NO}}
162166
clang_analyzer_isTainted(A.data); // expected-warning {{YES}}
163167
}
164168
} // namespace gh114270

0 commit comments

Comments
 (0)