Skip to content

Conversation

bader
Copy link
Contributor

@bader bader commented Aug 13, 2020

No description provided.

Copy link
Contributor

@mdtoguchi mdtoguchi left a comment

Choose a reason for hiding this comment

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

OK for Driver.

Copy link
Contributor

@premanandrao premanandrao left a comment

Choose a reason for hiding this comment

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

LGTM.

@bader
Copy link
Contributor Author

bader commented Aug 14, 2020

@intel/llvm-reviewers-runtime, ping.

Copy link
Contributor

@s-kanaev s-kanaev left a comment

Choose a reason for hiding this comment

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

Changes in RT tests LGTM.

@bader bader merged commit 077a507 into intel:sycl Aug 14, 2020
@bader bader deleted the option-rename branch August 14, 2020 09:13
jsji pushed a commit that referenced this pull request Mar 14, 2024
Translator accepting two formats currently - SPIR-V and SPT.
SPT is a textual format used internally for tests without any formal specification.
Parsing a textual format is slower than binary format and SPT also is less strict
in comparison to SPIRV. Because parsing in Translator is organized around
iostream interfaces and implementations are mostly shared, the binary
format is parsed much slower than it can be.
The commit starts an effort of splitting implementations to
make SPIR-V hot path faster and easier to read.
In addition to that from both implementations the Decoder class
is removed - it doesn't bring any important functionality which
couldn't be solved by few lines of explicit simple code.

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@d54f77c5dd695cc
Chenyang-L pushed a commit that referenced this pull request Feb 18, 2025
Change `urSuggestMaxCooperativeGroupCountExp` to accept ND size parameter
iclsrc pushed a commit that referenced this pull request Sep 27, 2025
d54f77c5 ("[NFC] Split of SPT and SPIR-V in header parsing (#2316)",
2024-03-11) made a copy of the error log, with the presumably unintended
consequence that errors are no longer propagated back to the SPIRVModule
itself.

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@3d58c69cf2f3704
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