Skip to content

[clang][test] Remove duplication from gcc toolchain test (NFC) #146487

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

frederik-h
Copy link
Contributor

Changes from Commit 40aab04 "[test] Migrate -gcc-toolchain with space separator to --gcc-toolchain=" made two previously different RUN lines equal.

Remove one RUN line.

Changes from Commit 40aab04
"[test] Migrate -gcc-toolchain with space separator to
--gcc-toolchain=" made two previously different RUN lines
equal.

Remove one RUN line.
@frederik-h frederik-h requested review from MaskRay and abrachet July 1, 2025 08:47
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' labels Jul 1, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 1, 2025

@llvm/pr-subscribers-clang

@llvm/pr-subscribers-clang-driver

Author: Frederik Harwath (frederik-h)

Changes

Changes from Commit 40aab04 "[test] Migrate -gcc-toolchain with space separator to --gcc-toolchain=" made two previously different RUN lines equal.

Remove one RUN line.


Full diff: https://github.com/llvm/llvm-project/pull/146487.diff

1 Files Affected:

  • (modified) clang/test/Driver/gcc-toolchain.cpp (-5)
diff --git a/clang/test/Driver/gcc-toolchain.cpp b/clang/test/Driver/gcc-toolchain.cpp
index 6d4ad417cfec6..a14e8d00af1ef 100644
--- a/clang/test/Driver/gcc-toolchain.cpp
+++ b/clang/test/Driver/gcc-toolchain.cpp
@@ -6,11 +6,6 @@
 // RUN:   --gcc-toolchain=%S/Inputs/ubuntu_14.04_multiarch_tree/usr -stdlib=libstdc++ --rtlib=libgcc --unwindlib=libgcc -no-pie 2>&1 | \
 // RUN:   FileCheck %s
 //
-// Additionally check that the legacy spelling of the flag works.
-// RUN: %clangxx %s -### --target=x86_64-linux-gnu --sysroot= \
-// RUN:   --gcc-toolchain=%S/Inputs/ubuntu_14.04_multiarch_tree/usr -stdlib=libstdc++ --rtlib=libgcc --unwindlib=libgcc -no-pie 2>&1 | \
-// RUN:   FileCheck %s
-//
 // Test for header search toolchain detection.
 // CHECK: "-internal-isystem"
 // CHECK: "[[TOOLCHAIN:[^"]+]]/usr/lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8"

@frederik-h
Copy link
Contributor Author

Merging without review as this change is trivial.

@frederik-h frederik-h merged commit f9413e1 into llvm:main Jul 1, 2025
10 checks passed
@frederik-h frederik-h deleted the gcc-toolchain-remove-duplicate-test branch July 1, 2025 14:17
rlavaee pushed a commit to rlavaee/llvm-project that referenced this pull request Jul 1, 2025
…146487)

Changes from Commit 40aab04 "[test]
Migrate -gcc-toolchain with space separator to --gcc-toolchain=" made
two previously different RUN lines equal.

Remove one RUN line.
rlavaee pushed a commit to rlavaee/llvm-project that referenced this pull request Jul 1, 2025
…146487)

Changes from Commit 40aab04 "[test]
Migrate -gcc-toolchain with space separator to --gcc-toolchain=" made
two previously different RUN lines equal.

Remove one RUN line.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants