[PATCH] D74104: Remove test dependency on the presence of an assembler
This revision was automatically updated to reflect the committed changes. Closed by commit rG54d82255fd27: Change tests to use -S instead of -c to work when an external assembler is used… (authored by dyung). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74104/new/ https://reviews.llvm.org/D74104 Files: clang/test/Analysis/scan-build/exclude_directories.test clang/test/Analysis/scan-build/html_output.test clang/test/Analysis/scan-build/plist_html_output.test clang/test/Analysis/scan-build/plist_output.test Index: clang/test/Analysis/scan-build/plist_output.test === --- clang/test/Analysis/scan-build/plist_output.test +++ clang/test/Analysis/scan-build/plist_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -plist -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -plist -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test plist output Index: clang/test/Analysis/scan-build/plist_html_output.test === --- clang/test/Analysis/scan-build/plist_html_output.test +++ clang/test/Analysis/scan-build/plist_html_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -plist-html -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -plist-html -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test combined plist and html output with -plist-html Index: clang/test/Analysis/scan-build/html_output.test === --- clang/test/Analysis/scan-build/html_output.test +++ clang/test/Analysis/scan-build/html_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test html output Index: clang/test/Analysis/scan-build/exclude_directories.test === --- clang/test/Analysis/scan-build/exclude_directories.test +++ clang/test/Analysis/scan-build/exclude_directories.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir %clang -c \ +RUN: %scan-build -o %t.output_dir %clang -S \ RUN: %S/Inputs/multidirectory_project/directory1/file1.c \ RUN: %S/Inputs/multidirectory_project/directory2/file2.c \ RUN: | FileCheck %s -check-prefix CHECK-NO-EXCLUDE @@ -21,7 +21,7 @@ // Only one issue should be found when directory1 is excluded. RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir --exclude directory1 %clang -c \ +RUN: %scan-build -o %t.output_dir --exclude directory1 %clang -S \ RUN: %S/Inputs/multidirectory_project/directory1/file1.c \ RUN: %S/Inputs/multidirectory_project/directory2/file2.c \ RUN: | FileCheck %s -check-prefix CHECK-EXCLUDE1 @@ -31,7 +31,7 @@ // When both directories are excluded, no issues should be reported. RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir --exclude directory1 --exclude directory2 %clang -c \ +RUN: %scan-build -o %t.output_dir --exclude directory1 --exclude directory2 %clang -S \ RUN: %S/Inputs/multidirectory_project/directory1/file1.c \ RUN: %S/Inputs/multidirectory_project/directory2/file2.c \ RUN: | FileCheck %s -check-prefix CHECK-EXCLUDE-BOTH Index: clang/test/Analysis/scan-build/plist_output.test === --- clang/test/Analysis/scan-build/plist_output.test +++ clang/test/Analysis/scan-build/plist_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -plist -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -plist -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test plist output Index: clang/test/Analysis/scan-build/plist_html_output.test === --- clang/test/Analysis/scan-build/plist_html_output.test +++ clang/test/Analysis/scan-build/plist_html_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -plist-html -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build
[PATCH] D74104: Remove test dependency on the presence of an assembler
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Interesting, indeed, thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74104/new/ https://reviews.llvm.org/D74104 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D74104: Remove test dependency on the presence of an assembler
dyung added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74104/new/ https://reviews.llvm.org/D74104 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D74104: Remove test dependency on the presence of an assembler
dyung created this revision. dyung added a reviewer: NoQ. dyung added a project: clang. Herald added a subscriber: Charusso. These tests currently call -c which requires an assembler, and will fail if your target defaults to an external assembler that is not present in the test environment. It doesn't seem like the assembler is actually needed, so just change the test to use -S instead. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D74104 Files: clang/test/Analysis/scan-build/exclude_directories.test clang/test/Analysis/scan-build/html_output.test clang/test/Analysis/scan-build/plist_html_output.test clang/test/Analysis/scan-build/plist_output.test Index: clang/test/Analysis/scan-build/plist_output.test === --- clang/test/Analysis/scan-build/plist_output.test +++ clang/test/Analysis/scan-build/plist_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -plist -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -plist -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test plist output Index: clang/test/Analysis/scan-build/plist_html_output.test === --- clang/test/Analysis/scan-build/plist_html_output.test +++ clang/test/Analysis/scan-build/plist_html_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -plist-html -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -plist-html -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test combined plist and html output with -plist-html Index: clang/test/Analysis/scan-build/html_output.test === --- clang/test/Analysis/scan-build/html_output.test +++ clang/test/Analysis/scan-build/html_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test html output Index: clang/test/Analysis/scan-build/exclude_directories.test === --- clang/test/Analysis/scan-build/exclude_directories.test +++ clang/test/Analysis/scan-build/exclude_directories.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir %clang -c \ +RUN: %scan-build -o %t.output_dir %clang -S \ RUN: %S/Inputs/multidirectory_project/directory1/file1.c \ RUN: %S/Inputs/multidirectory_project/directory2/file2.c \ RUN: | FileCheck %s -check-prefix CHECK-NO-EXCLUDE @@ -21,7 +21,7 @@ // Only one issue should be found when directory1 is excluded. RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir --exclude directory1 %clang -c \ +RUN: %scan-build -o %t.output_dir --exclude directory1 %clang -S \ RUN: %S/Inputs/multidirectory_project/directory1/file1.c \ RUN: %S/Inputs/multidirectory_project/directory2/file2.c \ RUN: | FileCheck %s -check-prefix CHECK-EXCLUDE1 @@ -31,7 +31,7 @@ // When both directories are excluded, no issues should be reported. RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -o %t.output_dir --exclude directory1 --exclude directory2 %clang -c \ +RUN: %scan-build -o %t.output_dir --exclude directory1 --exclude directory2 %clang -S \ RUN: %S/Inputs/multidirectory_project/directory1/file1.c \ RUN: %S/Inputs/multidirectory_project/directory2/file2.c \ RUN: | FileCheck %s -check-prefix CHECK-EXCLUDE-BOTH Index: clang/test/Analysis/scan-build/plist_output.test === --- clang/test/Analysis/scan-build/plist_output.test +++ clang/test/Analysis/scan-build/plist_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir %t.output_dir -RUN: %scan-build -plist -o %t.output_dir %clang -c %S/Inputs/single_null_dereference.c \ +RUN: %scan-build -plist -o %t.output_dir %clang -S %S/Inputs/single_null_dereference.c \ RUN: | FileCheck %s -check-prefix CHECK-STDOUT // Test plist output Index: clang/test/Analysis/scan-build/plist_html_output.test === --- clang/test/Analysis/scan-build/plist_html_output.test +++ clang/test/Analysis/scan-build/plist_html_output.test @@ -5,7 +5,7 @@ UNSUPPORTED: system-windows RUN: rm -rf %t.output_dir && mkdir