Skip to content

Fix test/transcoding/dbginfo-bug-on-bool-converts.ll test expectations#3879

Open
aobolensk wants to merge 1 commit into
KhronosGroup:mainfrom
aobolensk:align-tests
Open

Fix test/transcoding/dbginfo-bug-on-bool-converts.ll test expectations#3879
aobolensk wants to merge 1 commit into
KhronosGroup:mainfrom
aobolensk:align-tests

Conversation

@aobolensk

Copy link
Copy Markdown
Contributor

llvm/llvm-project#209232 fixed the SPIR-V backend selectIToF to preserve signedness (sitofp i1 true was miscompiled as 1.0 instead of -1.0), so the select now produces -1 rather than 1

llvm/llvm-project#209232 fixed the SPIR-V backend's selectIToF to preserve signedness (sitofp i1 true was miscompiled as 1.0 instead of -1.0), so the select now produces -1 rather than 1
@aobolensk

Copy link
Copy Markdown
Contributor Author

llvm-23 is still used on CI verification runs

@svenvh

svenvh commented Jul 17, 2026

Copy link
Copy Markdown
Member

llvm-23 is still used on CI verification runs

Indeed, we're waiting for new apt.llvm.org packages to appear before merging #3864

@idubinov idubinov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Qi-Ye-079

Copy link
Copy Markdown
Contributor

Can we merge this PR now? Since incoming PRs depend on this for CIs :)

@aobolensk

Copy link
Copy Markdown
Contributor Author

Can we merge this PR now? Since incoming PRs depend on this for CIs :)

I don't mind, but I don't have sufficient permissions to merge :)

@aobolensk

aobolensk commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

I don't mind, but I don't have sufficient permissions to merge :)

And they are kind of bundled together with #3864 (to actually fix all issues we have on the main llvm branch)

@Qi-Ye-079

Copy link
Copy Markdown
Contributor

@svenvh Can you merge this PR so that we can at least unblock all the in-tree CIs?

@svenvh

svenvh commented Jul 22, 2026

Copy link
Copy Markdown
Member

@svenvh Can you merge this PR so that we can at least unblock all the in-tree CIs?

We should not merge this before #3864 is merged; still waiting for a go-ahead there.

@YuriPlyakhin YuriPlyakhin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@YuriPlyakhin

Copy link
Copy Markdown
Contributor

@svenvh Can you merge this PR so that we can at least unblock all the in-tree CIs?

We should not merge this before #3864 is merged; still waiting for a go-ahead there.

@svenvh , I think this one could be merged before #3864, so that in-tree builds/tests would become green again.

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.

5 participants