[all-commits] [llvm/llvm-project] a7d7db: [NFC][analyzer] OOB test consolidation IV: rename ...

[Donát Nagy via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a7d7db71db2f4b1bca2484f2039ac57b01566a0e
      https://github.com/llvm/llvm-project/commit/a7d7db71db2f4b1bca2484f2039ac57b01566a0e
  Author: Donát Nagy <donat.nagy at ericsson.com>
  Date:   2025-03-11 (Tue, 11 Mar 2025)

  Changed paths:
    A clang/test/Analysis/ArrayBound/assumption-reporting.c
    A clang/test/Analysis/ArrayBound/assumptions.c
    A clang/test/Analysis/ArrayBound/brief-tests.c
    A clang/test/Analysis/ArrayBound/cplusplus.cpp
    A clang/test/Analysis/ArrayBound/verbose-tests.c
    R clang/test/Analysis/out-of-bounds-constraint-check.c
    R clang/test/Analysis/out-of-bounds-diagnostics.c
    R clang/test/Analysis/out-of-bounds-new.cpp
    R clang/test/Analysis/out-of-bounds-notes.c
    R clang/test/Analysis/out-of-bounds.c

  Log Message:
  -----------
  [NFC][analyzer] OOB test consolidation IV: rename files (#129697)

This commit finishes the reorganization of the tests for the checker
`security.ArrayBound`.

Previously these tests were all named `out-of-bounds-*` which was only
weakly connected to the checker name; this commit moves them to a
directory named after the checker (`ArrayBound`). I decided to use a
directory instead of the more common filename prefix ("poor man's
directory") system because it seems to be a more natural use of the
filesystem and there are already a few precedents for it.

I also added (or edited) comments at the beginning of each test file to
describe their purpose; and I added a single new testcase to highlight
that the assumption note tags can be added to reports by any checker.
(Previously all tests in the file triggered out-of-bounds reports to
reveal the note tags; but that was just for convenience.)



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications