Skip to content

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
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