Skip to content

Conversation

petrhosek
Copy link
Member

DTLTO is still incompatible with llvm-driver, but the test started passing after #159151.

DTLTO is still incompatible with llvm-driver, but the test started
passing after llvm#159151.
@llvmbot llvmbot added clang Clang issues not falling into any other category clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' labels Sep 17, 2025
@llvmbot
Copy link
Member

llvmbot commented Sep 17, 2025

@llvm/pr-subscribers-clang

@llvm/pr-subscribers-clang-driver

Author: Petr Hosek (petrhosek)

Changes

DTLTO is still incompatible with llvm-driver, but the test started passing after #159151.


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

1 Files Affected:

  • (modified) clang/test/Driver/DTLTO/dtlto.c (-3)
diff --git a/clang/test/Driver/DTLTO/dtlto.c b/clang/test/Driver/DTLTO/dtlto.c
index 66299c890e4b3..ebb9fc288e91e 100644
--- a/clang/test/Driver/DTLTO/dtlto.c
+++ b/clang/test/Driver/DTLTO/dtlto.c
@@ -1,8 +1,5 @@
 // REQUIRES: lld
 
-/// https://github.com/llvm/llvm-project/issues/159125.
-// XFAIL: llvm-driver
-
 /// Check DTLTO options are forwarded to the linker.
 
 /// Check that options are forwarded as expected with --thinlto-distributor=.

@bd1976bris
Copy link
Collaborator

@petrhosek. Thanks for putting this up and apologies for the disruption to you. There was another failure due to the regex's being too greedy. Can you please review PR #159418 in preference to this. Thanks.

Copy link
Collaborator

@bd1976bris bd1976bris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully, we can land PR #159418 in preference.

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.

3 participants