[clang] [NFC][analyzer] OOB test consolidation III: 'outofbound' tests (PR #128508)

Endre Fülöp via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 28 05:53:37 PST 2025


https://github.com/gamesh411 approved this pull request.

This is a nice cleanup change. I have checked the test cases and the reasoning behind the removals.
It is also welcome that these reasons are documented, at least in the PR.
@NagyDonat  What do you think about making these reasoning snippets available in the commit message?
I am not sure that it is enough for them to only exist in the PR comments, but if you say that it would be unnecessary verbose to have the gist of them in the commit message, that is fine as well.

https://github.com/llvm/llvm-project/pull/128508


More information about the cfe-commits mailing list