[clang] c80b99d - [Sema][NFC] Move two misplaced uninit tests to clang/test/SemaCXX (#128013)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 24 03:44:03 PST 2025
Author: DonĂ¡t Nagy
Date: 2025-02-24T12:44:00+01:00
New Revision: c80b99d98ad0c7a3c185b4f09d2b60affb34cad1
URL: https://github.com/llvm/llvm-project/commit/c80b99d98ad0c7a3c185b4f09d2b60affb34cad1
DIFF: https://github.com/llvm/llvm-project/commit/c80b99d98ad0c7a3c185b4f09d2b60affb34cad1.diff
LOG: [Sema][NFC] Move two misplaced uninit tests to clang/test/SemaCXX (#128013)
Because they are the last two remaining test files that appeared under
`clang/test/Analysis` but were unrelated to the clang static analyzer.
For background see the following discourse thread:
https://discourse.llvm.org/t/taking-ownership-of-clang-test-analysis/84689/2
I placed them in in `clang/test/SemaCXX` because they are testing the
`-Wuninitialized` warning family and the other tests of this feature can
be found there (or in `Sema`, `SemaObjC` depending on the language).
Note that `clang/test/Analysis` contains many other test files named
`uninit-*`, but those test the uninitialized value handling of the clang
static analyzer, which is independent of the (non-path-sensitive)
compiler warnings that are tested in the two files that I'm moving.
Also note that the implementation of the `-Wuninitialized`-like warnings
relies on the source files `clang/lib/Analysis/UninitializedValues.cpp`
and `clang/lib/Sema/AnalysisBasedWarnings.cpp`, and this would
theoretically justify leaving their tests in the "Analysis" directory;
but in practice the "Analysis" directory is almost exclusively used by
the static analyzer, so I still decided to relocate these two tests for
the sake of consistency.
Added:
clang/test/SemaCXX/uninit-asm-goto.cpp
clang/test/SemaCXX/uninit-sometimes.cpp
Modified:
Removed:
clang/test/Analysis/uninit-asm-goto.cpp
clang/test/Analysis/uninit-sometimes.cpp
################################################################################
diff --git a/clang/test/Analysis/uninit-asm-goto.cpp b/clang/test/SemaCXX/uninit-asm-goto.cpp
similarity index 100%
rename from clang/test/Analysis/uninit-asm-goto.cpp
rename to clang/test/SemaCXX/uninit-asm-goto.cpp
diff --git a/clang/test/Analysis/uninit-sometimes.cpp b/clang/test/SemaCXX/uninit-sometimes.cpp
similarity index 100%
rename from clang/test/Analysis/uninit-sometimes.cpp
rename to clang/test/SemaCXX/uninit-sometimes.cpp
More information about the cfe-commits
mailing list