Skip to content
This repository has been archived by the owner on Oct 9, 2019. It is now read-only.

OpSource should allow optional string operand for source code #12

Open
yxsamliu opened this issue Dec 1, 2015 · 0 comments
Open

OpSource should allow optional string operand for source code #12

yxsamliu opened this issue Dec 1, 2015 · 0 comments

Comments

@yxsamliu
Copy link
Contributor

yxsamliu commented Dec 1, 2015

No description provided.

@yxsamliu yxsamliu changed the title OpSource will allow optional string operand for source code OpSource should allow optional string operand for source code Dec 1, 2015
echuraev added a commit to echuraev/SPIRV-LLVM that referenced this issue Sep 26, 2016
@echuraev
Fixed problem with OpSpecConstantOp.
b1274b7 
In one of last change of spirv spec was added following change:
"Public SPIRV-Headers issues KhronosGroup#12 and KhronosGroup#13 and Khronos SPIR-V issue KhronosGroup#65: Allow OpVariable as an initializer for another OpVariable instruction or the Base of an OpSpecConstantOp with an AccessChain opcode."
In this way, OpVariable can be initialize before OpSpecConstantOp. To do this, it's necessary add OpSpecConstantOp in one container with OpVariable in TopologicalSort. After this change, OpSpecConstantOp and OpVariable instructions will be add to spirv file in the right order.
@echuraev echuraev mentioned this issue Sep 26, 2016
Merged
yxsamliu pushed a commit that referenced this issue Sep 29, 2016
@echuraev @yxsamliu
OpVariable in OpSpecConstantOp. (#186)
7771f22 
* Fixed problem with OpSpecConstantOp.
In one of last change of spirv spec was added following change:
"Public SPIRV-Headers issues #12 and #13 and Khronos SPIR-V issue #65: Allow OpVariable as an initializer for another OpVariable instruction or the Base of an OpSpecConstantOp with an AccessChain opcode."
In this way, OpVariable can be initialize before OpSpecConstantOp. To do this, it's necessary add OpSpecConstantOp in one container with OpVariable in TopologicalSort. After this change, OpSpecConstantOp and OpVariable instructions will be add to spirv file in the right order.

* Renamed InstructionVec to ConstAndVarVec and fixed indentation
@yxsamliu yxsamliu mentioned this issue May 18, 2017
Merged
3 tasks
robclark pushed a commit to freedreno/SPIRV-LLVM that referenced this issue Jan 26, 2018
@echuraev @yxsamliu
OpVariable in OpSpecConstantOp. (KhronosGroup#186)
c657097 
* Fixed problem with OpSpecConstantOp.
In one of last change of spirv spec was added following change:
"Public SPIRV-Headers issues KhronosGroup#12 and KhronosGroup#13 and Khronos SPIR-V issue KhronosGroup#65: Allow OpVariable as an initializer for another OpVariable instruction or the Base of an OpSpecConstantOp with an AccessChain opcode."
In this way, OpVariable can be initialize before OpSpecConstantOp. To do this, it's necessary add OpSpecConstantOp in one container with OpVariable in TopologicalSort. After this change, OpSpecConstantOp and OpVariable instructions will be add to spirv file in the right order.

* Renamed InstructionVec to ConstAndVarVec and fixed indentation
AlexeySotkin pushed a commit to AlexeySotkin/SPIRV-LLVM that referenced this issue Mar 12, 2018
@echuraev @yxsamliu
OpVariable in OpSpecConstantOp. (KhronosGroup#186)
6826048 
* Fixed problem with OpSpecConstantOp.
In one of last change of spirv spec was added following change:
"Public SPIRV-Headers issues KhronosGroup#12 and KhronosGroup#13 and Khronos SPIR-V issue KhronosGroup#65: Allow OpVariable as an initializer for another OpVariable instruction or the Base of an OpSpecConstantOp with an AccessChain opcode."
In this way, OpVariable can be initialize before OpSpecConstantOp. To do this, it's necessary add OpSpecConstantOp in one container with OpVariable in TopologicalSort. After this change, OpSpecConstantOp and OpVariable instructions will be add to spirv file in the right order.

* Renamed InstructionVec to ConstAndVarVec and fixed indentation
AlexeySotkin pushed a commit to AlexeySotkin/SPIRV-LLVM that referenced this issue Apr 3, 2018
@echuraev @yxsamliu
OpVariable in OpSpecConstantOp. (KhronosGroup#186)
12ecce2 
* Fixed problem with OpSpecConstantOp.
In one of last change of spirv spec was added following change:
"Public SPIRV-Headers issues KhronosGroup#12 and KhronosGroup#13 and Khronos SPIR-V issue KhronosGroup#65: Allow OpVariable as an initializer for another OpVariable instruction or the Base of an OpSpecConstantOp with an AccessChain opcode."
In this way, OpVariable can be initialize before OpSpecConstantOp. To do this, it's necessary add OpSpecConstantOp in one container with OpVariable in TopologicalSort. After this change, OpSpecConstantOp and OpVariable instructions will be add to spirv file in the right order.

* Renamed InstructionVec to ConstAndVarVec and fixed indentation
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yxsamliu