diff options
| author | Balazs Benics <benicsbalazs@gmail.com> | 2025-02-17 11:12:55 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-02-17 11:12:55 +0100 |
| commit | f378e52ed3c6f8da4973f97f1ef043c2eb0da721 (patch) | |
| tree | 1c6125829ef1f9d8a4ba95ac7287e83bd62e7ab7 /llvm/docs/tutorial | |
| parent | 501c77da6024959438c92f33bd997fe6f39e6b6c (diff) | |
| download | llvm-f378e52ed3c6f8da4973f97f1ef043c2eb0da721.zip llvm-f378e52ed3c6f8da4973f97f1ef043c2eb0da721.tar.gz llvm-f378e52ed3c6f8da4973f97f1ef043c2eb0da721.tar.bz2 | |
[clang][analysis] Fix flaky clang/test/Analysis/live-stmts.cpp test (2nd attempt) (#127406)
In my previous attempt (#126913) of fixing the flaky case was on a good
track when I used the begin locations as a stable ordering. However, I
forgot to consider the case when the begin locations are the same among
the Exprs.
In an `EXPENSIVE_CHECKS` build, arrays are randomly shuffled prior to
sorting them. This exposed the flaky behavior much more often basically
breaking the "stability" of the vector - as it should.
Because of this, I had to revert the previous fix attempt in #127034.
To fix this, I use this time `Expr::getID` for a stable ID for an Expr.
Hopefully fixes #126619
Hopefully fixes #126804
Diffstat (limited to 'llvm/docs/tutorial')
0 files changed, 0 insertions, 0 deletions