[clang] [Sema][NFC] Move two misplaced uninit tests to clang/test/SemaCXX (PR #128013)
via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 20 07:02:09 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: DonĂ¡t Nagy (NagyDonat)
<details>
<summary>Changes</summary>
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.
---
Full diff: https://github.com/llvm/llvm-project/pull/128013.diff
2 Files Affected:
- (renamed) clang/test/SemaCXX/uninit-asm-goto.cpp ()
- (renamed) clang/test/SemaCXX/uninit-sometimes.cpp ()
``````````diff
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
``````````
</details>
https://github.com/llvm/llvm-project/pull/128013
More information about the cfe-commits
mailing list