Author: Donát Nagy Date: 2026-09-17T13:52:22-04:00 New Revision: 6d1d34cfb6dba451912371f6ba41d759d77bdf7c
URL: https://github.com/llvm/llvm-project/commit/6d1d34cfb6dba451912371f6ba41d759d77bdf7c DIFF: https://github.com/llvm/llvm-project/commit/6d1d34cfb6dba451912371f6ba41d759d77bdf7c.diff LOG: [SSAF] Mark two failing tests as unsupported (#223976) As a developer of the clang static analyzer, I'm using the build target `check-clang-analysis` [1] to build and test my changes. This workflow was recently broken by two new SSAF testcases under `Analysis/Scalable` that fail on my machine (where I don't build any SSAF-specific targets and perhaps don't have some SSAF-specific tools). In both of these tests a `clang-apply-replacements` RUN line fails with `clang-apply-replacements: symbol lookup error: clang-apply-replacements: undefined symbol: _ZN4llvm2cl6OptionC2ENS0_18NumOccurrencesFlagENS0_12OptionHidden` These two files were added by 77a6638e4f61e1827001cce202d8f785815a341b and 4b08871f7ffe6ad818147145f8b6103e6c9b2c95. In this commit I'm marking these two files as unsupported, because I'm not familiar with SSAF and don't have the capacity to troubleshoot them. (I'm not using XFAIL because they are presumably passing on some other machines.) [1] Before the introduction of SSAF and `Analysis/Scalable`, this target only invoked tests for the clang static analyzer. See also: https://discourse.llvm.org/t/taking-ownership-of-clang-test-analysis/84689 ------- I already mentioned this issue on the PR that has (re-)added one of the failing test files [in a github comment](https://github.com/llvm/llvm-project/pull/220124#issuecomment-5527847470) but it was probably overlooked among the many github notifications. Added: Modified: clang/test/Analysis/Scalable/source-edit-generation/cpp-bounded-buffers-replacements.cpp clang/test/Analysis/Scalable/source-edit-generation/multi-tu-edits-merge.cpp Removed: ################################################################################ diff --git a/clang/test/Analysis/Scalable/source-edit-generation/cpp-bounded-buffers-replacements.cpp b/clang/test/Analysis/Scalable/source-edit-generation/cpp-bounded-buffers-replacements.cpp index 3c6d8211b85de..de33fa1be44a6 100644 --- a/clang/test/Analysis/Scalable/source-edit-generation/cpp-bounded-buffers-replacements.cpp +++ b/clang/test/Analysis/Scalable/source-edit-generation/cpp-bounded-buffers-replacements.cpp @@ -3,6 +3,7 @@ // 'clang-apply-replacements' tool is available, it also applies the // YAML and checks the resulting rewritten source. +// UNSUPPORTED: target={{.*}} // RUN: rm -rf %t && mkdir -p %t // RUN: split-file %s %t diff --git a/clang/test/Analysis/Scalable/source-edit-generation/multi-tu-edits-merge.cpp b/clang/test/Analysis/Scalable/source-edit-generation/multi-tu-edits-merge.cpp index 7057a2a6ed078..b366c3741a356 100644 --- a/clang/test/Analysis/Scalable/source-edit-generation/multi-tu-edits-merge.cpp +++ b/clang/test/Analysis/Scalable/source-edit-generation/multi-tu-edits-merge.cpp @@ -1,6 +1,8 @@ // Tests clang-ssaf-src-edit-merge against real multi-TU source. // Specifically, this file tests non-conflicting merges across multiple TUs. +// UNSUPPORTED: target={{.*}} + // RUN: rm -rf %t && mkdir -p %t // RUN: split-file %s %t _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
