Skip to content

Fix windows nightly failures#729

Merged
quic-seaswara merged 1 commit intoqualcomm:mainfrom
Ris-Bali:fix_windows_dev_null_failure
Jan 23, 2026
Merged

Fix windows nightly failures#729
quic-seaswara merged 1 commit intoqualcomm:mainfrom
Ris-Bali:fix_windows_dev_null_failure

Conversation

@Ris-Bali
Copy link
Contributor

This patch fixes windows nightly failures related to /dev/null path issue and multiconfig build related path issue.

This patch fixes windows nightly failures related
to `/dev/null` path issue and multiconfig build
related path issue.

Signed-off-by: Rishabh Bali <rbali@qti.qualcomm.com>
@quic-seaswara quic-seaswara self-requested a review January 23, 2026 17:34
@quic-seaswara quic-seaswara merged commit 58e6823 into qualcomm:main Jan 23, 2026
3 checks passed
@@ -1,3 +1,4 @@
#UNSUPPORTED: windows
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is it not working on windows? Is there a plan to fix the root cause and reenable it?

; RUN: %link %linkopts --plugin-opt=debug-pass-manager -o /dev/null %t.o 2>&1 | FileCheck %s
; RUN: %link %linkopts --lto-debug-pass-manager -o /dev/null %t.o 2>&1 | FileCheck %s
; RUN: %link %linkopts --plugin-opt=debug-pass-manager -o %t.temp %t.o 2>&1 | FileCheck %s
; RUN: %link %linkopts --lto-debug-pass-manager -o %t.temp %t.o 2>&1 | FileCheck %s
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we have other tests that use /dev/null as output path?

@quic-seaswara
Copy link
Contributor

Yes, I have asked to handle both of these as follow ups.

/dev/null needs to be probably regarded specially on windows as well.

Rishabh, please open follow up tickets.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants