Skip to content
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

Cherrypick fixes for 6223 and 6168 to release branch. #6340

Open
wants to merge 6 commits into
base: release-1.8.2403
Choose a base branch
from

Conversation

python3kgae
Copy link
Contributor

Other PRs are for lit which enables dxil version check so the tests could be skipped for older dxil.dll.

More testing of node shader objects passed as arguments.
noinline and export still not work, added as XFail.

For microsoft#6124

(cherry picked from commit 13609db)
…rdPtrAndUpdateUsers (microsoft#6314)

In EmitGetNodeRecordPtrAndUpdateUsers, the type will be mutated. And the
GEP user of the RecordPtr will be merged at same time. This make things
complex because the GEP index need to be updated since type is mutated.

To make things easier, merge the GepUse after mutate type.

Fixes microsoft#6223

(cherry picked from commit 823125b)
Add lit.local.cfg to set unsupported when not dxil-1-8.

(cherry picked from commit 76f00d0)
Replace assert on illegal DXIL op with return illegal value. Check the
illegal cases in validation.

Fixes microsoft#6168

(cherry picked from commit 5dee81c)
This will allow lit test check dxil version to avoid running on
unsupported dxil.dll.

(cherry picked from commit 4c4a588)
Check dxc version and dxil version separately.

(cherry picked from commit 5e658ac)
@python3kgae python3kgae requested a review from a team as a code owner February 21, 2024 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New
Development

Successfully merging this pull request may close these issues.

None yet

1 participant