David Neto [Thu, 4 Aug 2016 18:57:09 +0000 (14:57 -0400)]
Only check def dominance of reachable uses
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/295
David Neto [Tue, 2 Aug 2016 21:48:16 +0000 (17:48 -0400)]
Update CHANGES to reflect INotEqual update
qining [Tue, 26 Jul 2016 21:52:06 +0000 (17:52 -0400)]
Allow OpINotEqual in OpSpecConstantOp inst
Pendding for now until the spec is fixed.
David Neto [Tue, 2 Aug 2016 15:59:10 +0000 (11:59 -0400)]
Never generate % for a friendly disassembled Id
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/290
David Neto [Fri, 29 Jul 2016 22:05:02 +0000 (18:05 -0400)]
Add disabled test for phi variable def-use check
Disabled because doing a proper check requires much more
code, and it can wait.
David Neto [Fri, 29 Jul 2016 21:53:46 +0000 (17:53 -0400)]
Avoid checking def-use dominance for OpPhi value operands
The def-use dominance checker doesn't have enough info to know
that a particular use is in an OpPhi, so skip tracking those uses
for now. Add a TODO to do a proper OpPhi variable-argument check
in the future.
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/286
Lei Zhang [Wed, 27 Jul 2016 21:37:04 +0000 (17:37 -0400)]
Optimization: Add def use analysis.
Lei Zhang [Thu, 28 Jul 2016 13:09:00 +0000 (09:09 -0400)]
Use default copy/move constructors/assignments for Instruction.
Lei Zhang [Wed, 27 Jul 2016 21:31:23 +0000 (17:31 -0400)]
Allow missing memory model instructions in modules.
David Neto [Wed, 27 Jul 2016 21:02:22 +0000 (17:02 -0400)]
Fix infinite loop in dominance calculation.
Ensure the dominance calculation visits all nodes in the CFG.
The successor list of the pseudo-entry node is augmented with
a single node in each cycle that otherwise would not be visited.
Similarly, the predecssors list of the pseduo-exit node is augmented
with the a single node in each cycle that otherwise would not
be visited.
Pulls DepthFirstSearch out so it's accessible outside of the dominator
calculation.
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/279
Qining [Thu, 28 Jul 2016 19:33:16 +0000 (15:33 -0400)]
Merge pull request #284 from Qining/fix-freeze-spec-const-pass-test
Fix a test failure in pass manager test
qining [Thu, 28 Jul 2016 19:30:37 +0000 (15:30 -0400)]
Fix a test failure in pass manager test
Change "Null" to "null" in pass manager test
qining [Tue, 26 Jul 2016 16:11:03 +0000 (12:11 -0400)]
Freeze spec constants to their default values
Add a pass to freeze spec constants to their default values. This pass does
not fold the frozen spec constants and does not handle SpecConstantOp
instructions and SpecConstantComposite instructions.
qining [Wed, 27 Jul 2016 21:22:45 +0000 (17:22 -0400)]
Add an utility: in-place string replacement
qining [Wed, 27 Jul 2016 14:19:39 +0000 (10:19 -0400)]
Use friendly disassembly output
Umar Arshad [Wed, 13 Jul 2016 22:57:52 +0000 (18:57 -0400)]
Check definitions appear in dominator of use
Also address use and def of ID in same instruction
Umar Arshad [Fri, 8 Jul 2016 13:44:10 +0000 (09:44 -0400)]
Refactor IDs definition and use tracking
* Creates an ID class which manages definition and use of IDs
* Moved tracking code from validate.cpp to validate_id.cpp
* Rename and combine SsaPass and ProcessIds into IdPass
* Remove module dependency in Function
Umar Arshad [Fri, 22 Jul 2016 20:27:21 +0000 (16:27 -0400)]
Fixes segfault for loops without back-edges
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/270
Lei Zhang [Tue, 19 Jul 2016 19:04:43 +0000 (15:04 -0400)]
Start v2016.2.
Lei Zhang [Tue, 19 Jul 2016 19:03:52 +0000 (15:03 -0400)]
Finalize v2016.1.
David Neto [Fri, 8 Jul 2016 18:29:52 +0000 (14:29 -0400)]
Disassembling: map IDs to friendly names.
Add a FriendlyNameMapper to deduce friendly names for IDs based on
OpName, type structure, etc.
Lei Zhang [Mon, 18 Jul 2016 15:01:36 +0000 (11:01 -0400)]
Track changes in SPIRV-Headers for SPIR-V 1.1 rev 2.
* The `Input` StorageClass doesn't require the `Shader` capability
anymore.
* The `Sampled1D` and `SampledBuffer` capabilities don't require
the `Shader` capability anymore. So they do not indirectly
depend on the `Matrix` capability. So are the `Image1D` and
`ImageBuffer` capabilities, which depend on `Sampled1D` and
`SampledBuffer`.
A new GLSL grammar file is uploaded for SPIR-V 1.1, but it's the
same as the existing one for SPIR-V 1.0.
Now tracking commit
3814effb879ab5a98a7b9288a4b4c7849d2bc8ac in
SPIRV-Headers.
David Neto [Thu, 7 Jul 2016 21:03:22 +0000 (17:03 -0400)]
Turn off ClipDistance CullDistance cap checks for Vulkan
Turn them off until resolution of the debate over how they should be checked.
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/261
Lei Zhang [Thu, 7 Jul 2016 19:04:06 +0000 (15:04 -0400)]
Mention the steps to pull dependencies when building in README.
David Neto [Tue, 5 Jul 2016 14:21:21 +0000 (10:21 -0400)]
Test v1.1 loop dependency support
For DependencyInfinite and DependencyLength, test
that they don't require a capability to be turned on.
Also, that they are assembled, binary parsed, and disassembled
correctly.
David Neto [Mon, 4 Jul 2016 19:13:08 +0000 (15:13 -0400)]
Start v2016.1-dev
David Neto [Thu, 2 Jun 2016 19:29:13 +0000 (15:29 -0400)]
Finalize v2016.0
Lei Zhang [Thu, 30 Jun 2016 19:44:36 +0000 (15:44 -0400)]
DRY file writing code in tools.
Lei Zhang [Thu, 30 Jun 2016 18:24:04 +0000 (14:24 -0400)]
DRY file reading code in tools.
Lei Zhang [Thu, 30 Jun 2016 17:56:34 +0000 (13:56 -0400)]
Rework CMake configuration for tools.
Lei Zhang [Wed, 29 Jun 2016 20:16:03 +0000 (16:16 -0400)]
Add the spirv-opt command line tool.
Lei Zhang [Tue, 28 Jun 2016 18:58:17 +0000 (14:58 -0400)]
Add Pass, PassManager, and StripDebugInfoPass.
David Neto [Tue, 28 Jun 2016 21:25:28 +0000 (17:25 -0400)]
Avoid capability checks on ID values. They are not literals
Works around issue 248 by weakening the test:
https://github.com/KhronosGroup/SPIRV-Tools/issues/248
The validator should try to track (32-bit) constant values, and then
for capability checks on IDs, check the referenced value, not the
raw ID number.
Lei Zhang [Tue, 28 Jun 2016 17:24:10 +0000 (13:24 -0400)]
Add tests for the cpp interface.
Lei Zhang [Tue, 28 Jun 2016 14:23:13 +0000 (10:23 -0400)]
Promote ir namespace and create draft libspirv.{h|c}pp.
Lei Zhang [Sun, 22 May 2016 18:17:39 +0000 (14:17 -0400)]
Add IrBuilder for constructing SPIR-V in-memory representation.
Lei Zhang [Mon, 27 Jun 2016 19:06:41 +0000 (15:06 -0400)]
Add an option to spvBinaryToText() to ignore header output.
Umar Arshad [Sat, 25 Jun 2016 03:45:25 +0000 (23:45 -0400)]
Fix several violations of the style guide
Lei Zhang [Sun, 22 May 2016 18:11:24 +0000 (14:11 -0400)]
Add classes for representing SPIR-V language constructs in memory.
Lei Zhang [Fri, 24 Jun 2016 19:48:34 +0000 (15:48 -0400)]
Travis CI: bump OS X and XCode version.
David Neto [Fri, 24 Jun 2016 06:14:16 +0000 (02:14 -0400)]
Use pseudo entry and pseudo exit blocks for dominance.
For dominance calculations we use an "augmented" CFG
where we always add a pseudo-entry node that is the predecessor
in the augmented CFG to any nodes that have no predecessors in the
regular CFG. Similarly, we add a pseudo-exit node that is the
predecessor in the augmented CFG that is a successor to any
node that has no successors in the regular CFG.
Pseudo entry and exit blocks live in the Function object.
Fixes a subtle problem where we were implicitly creating
the block_details for the pseudo-exit node since it didn't
appear in the idoms map, and yet we referenced it. In such a case the
contents of the block details could be garbage, or zero-initialized.
That sometimes caused incorrect calculation of immediate dominators
and post-dominators. For example, on a debug build where the details
could be zero-initialized, the dominator of an unreachable block would
be given as the pseudo-exit node. Bizarre.
Also, enforce the rule that you must have an OpFunctionEnd to close off
the last function.
Lei Zhang [Tue, 21 Jun 2016 22:29:36 +0000 (18:29 -0400)]
Use arguments for add_spvtools_unittest.
Lei Zhang [Wed, 22 Jun 2016 14:53:41 +0000 (10:53 -0400)]
Appveyor: Pull SPIRV-Headers for building and update README link.
Umar Arshad [Sat, 4 Jun 2016 01:24:24 +0000 (21:24 -0400)]
Validator structured flow checks: back-edge, constructs
Skip structured control flow chekcs for non-shader capability.
Fix infinite loop in dominator algorithm when there's an
unreachable block.
Lei Zhang [Tue, 21 Jun 2016 21:27:27 +0000 (17:27 -0400)]
Ignore Vim swap files.
David Neto [Tue, 14 Jun 2016 20:41:27 +0000 (16:41 -0400)]
Fix ExtInst parsing: no IdRef* at end
The operands following the extended instruction literal
number are determined by the extended instruction itself.
So drop the zero-or-more IdRef pattern at the end of OpExtInst.
It's arguable whether this should actually be a grammar fix. I've
chosen to patch this in SPIRV-Tools instead of in the grammar file.
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/233
Also fix two test cases for OpenCL extended instructions. These
errors of supplying too many operands are now detected.
David Neto [Tue, 14 Jun 2016 15:45:35 +0000 (11:45 -0400)]
Fix OpenCL pow and pown arguments
Spec fix appeared in SPIR-V 1.0 Rev 2:
https://www.khronos.org/registry/spir-v/specs/1.0/OpenCL.ExtendedInstructionSet.100.html#_changes_from_version_1_0_revision_1
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/143
David Neto [Mon, 13 Jun 2016 21:26:09 +0000 (17:26 -0400)]
In physical addressing, functions can return pointers
Fixes https://github.com/KhronosGroup/SPIRV-Tools/issues/229
Lei Zhang [Thu, 14 Apr 2016 18:05:53 +0000 (14:05 -0400)]
Use SPIR-V headers from the KhronosGroup/SPIRV-Headers repo.
David Neto [Fri, 10 Jun 2016 12:43:12 +0000 (08:43 -0400)]
std::function needs <functional>. Fixes Android build
David Neto [Fri, 10 Jun 2016 10:41:20 +0000 (06:41 -0400)]
Avoid copy of work list argument: make it const ref
Umar Arshad [Sat, 4 Jun 2016 01:14:22 +0000 (21:14 -0400)]
Refactor depth first traversal to be more generic
Refactor the way the post order vector is created. This new method
will allow for the extraction of backedges and create the post order
vector in one pass.
Umar Arshad [Thu, 9 Jun 2016 19:51:39 +0000 (15:51 -0400)]
Fix diagnostic message for layout errors
Umar Arshad [Thu, 2 Jun 2016 22:51:05 +0000 (18:51 -0400)]
Split validate_types file into multiple classes
Creates separate files for the ValidationState, Function and
BasicBlock classes.
Umar Arshad [Wed, 1 Jun 2016 22:22:57 +0000 (18:22 -0400)]
Remove redundant in_block function from Function
Same test can be done through the get_current_block function
Dejan Mircevski [Sat, 4 Jun 2016 01:46:19 +0000 (21:46 -0400)]
Avoid accessing invalid iterator.
Umar Arshad [Wed, 1 Jun 2016 22:05:28 +0000 (18:05 -0400)]
Additional CFG tests
* Empty function
* Single block loops
* Nested loops
* Nested selection
David Neto [Thu, 2 Jun 2016 19:32:28 +0000 (15:32 -0400)]
Fix CHANGES file version parsing. -dev is optional
Dejan Mircevski [Thu, 2 Jun 2016 17:13:04 +0000 (13:13 -0400)]
Make googletest regexes portable.
On Windows, test::MatchesRegex() doesn't support brackets. On Linux, \d
doesn't work. :(
Umar Arshad [Wed, 16 Mar 2016 21:20:02 +0000 (17:20 -0400)]
Control Flow Graph Validation
Lei Zhang [Sun, 22 May 2016 17:48:08 +0000 (13:48 -0400)]
Create a function for adding unittests in CMake.
Also remove unnecessary main() function for unittests.
Lei Zhang [Thu, 26 May 2016 01:01:54 +0000 (21:01 -0400)]
Travis CI: Use ctest to run all the tests.
Lei Zhang [Tue, 17 May 2016 13:02:45 +0000 (09:02 -0400)]
Add Appveyor configuration for Windows build.
Dejan Mircevski [Fri, 13 May 2016 02:06:28 +0000 (22:06 -0400)]
Test ABGR in both 1.0 and 1.1.
Dejan Mircevski [Wed, 11 May 2016 19:21:10 +0000 (15:21 -0400)]
Remove needless include dir for spirv-dis.
Dejan Mircevski [Wed, 11 May 2016 17:12:59 +0000 (13:12 -0400)]
Test ABGR image channel order.
Update include/spirv/spirv.h from SPIRV-Headers.
Dejan Mircevski [Wed, 11 May 2016 16:40:59 +0000 (12:40 -0400)]
Test OpModuleProcessed.
Dejan Mircevski [Wed, 11 May 2016 14:34:27 +0000 (10:34 -0400)]
Remove done TODOs in OpNameTest.
Simplify the instantiation by using Values() instead of ValuesIn().
Run clang-format.
Dejan Mircevski [Tue, 10 May 2016 21:02:41 +0000 (17:02 -0400)]
Test SpecId validation.
Dejan Mircevski [Wed, 11 May 2016 13:48:52 +0000 (09:48 -0400)]
Add --target-env.
commit
7648187c344cfc7847607acb2a6fa5606a3d10c1
Author: Dejan Mircevski <deki@google.com>
Date: Thu Apr 21 15:44:07 2016 -0400
Turn --vulkan into --target-env.
Dejan Mircevski [Tue, 10 May 2016 03:15:46 +0000 (23:15 -0400)]
Add tests for MaxByteOffset.
Jamie Madill [Fri, 29 Apr 2016 18:36:00 +0000 (14:36 -0400)]
Add PipeStorage tests.
Jamie Madill [Fri, 29 Apr 2016 18:36:00 +0000 (14:36 -0400)]
Fix several warnings exposed in MSVS 2015.
diagnostic.cpp:
- unreachable code
operand.cpp
- conversion between int and uint32_t
- unreachable code
hex_float.h:
- conversion from 'const int' to 'unsigned int'
- unreachable code
validate_id.cpp
- forcing value to bool 'true' or 'false'
validate_types.cpp:
- forcing value to bool 'true' or 'false'
Florian Ziesche [Wed, 2 Mar 2016 21:17:54 +0000 (22:17 +0100)]
bitcast fixes
* ValidationState_t and idUsage now store the addressing model and memory model of the SPIR-V module (this is necessary for certain instructions that need different checks depending on if the logical or physical addressing model is used)
* removed SpvOpPtrAccessChain and SpvOpInBoundsPtrAccessChain from spvOpcodeIsPointer again as these are disallowed in logical addressing mode and only allowed in physical addressing mode (which doesn't use/need spvOpcodeIsPointer in the first place)
* added SpvOpImageTexelPointer and SpvOpCopyObject to spvOpcodeIsPointer
* OpLoad/OpStore now only check if the used pointer operand originated from a valid pointer producing opcode in logical addressing mode (as per 2.16.1)
* moved bitcast pointer tests to the kernel / physical addressing model part (+cleanup)
* renamed spvOpcodeIsPointer to spvOpcodeReturnsLogicalPointer to clarify this function is only meant to be used with the logical addressing model
Dejan Mircevski [Wed, 27 Apr 2016 20:22:15 +0000 (16:22 -0400)]
Add v1.1 tests for OpLoopMerge.
Dejan Mircevski [Fri, 29 Apr 2016 17:40:01 +0000 (13:40 -0400)]
Add v1.1 capabilities validation.
Refactor the ValidateCapability test fixture.
Explain the meaning of test parameters. Factor out methods for
convenience and readability. DRY v1.0 and v1.1 tests.
Dejan Mircevski [Wed, 27 Apr 2016 21:04:37 +0000 (17:04 -0400)]
Add basic Initializer/Finalizer tests.
Lei Zhang [Wed, 27 Apr 2016 20:47:13 +0000 (16:47 -0400)]
Use a single definition of ARRAY_SIZE to simplify code.
Dejan Mircevski [Wed, 27 Apr 2016 19:41:58 +0000 (15:41 -0400)]
Add OpSizeOf tests.
Dejan Mircevski [Tue, 26 Apr 2016 17:03:28 +0000 (13:03 -0400)]
Refactor OpExecutionModeTest and apply it on v1.1.
Previously v1.1 testing was dead.
Dejan Mircevski [Tue, 26 Apr 2016 16:16:00 +0000 (12:16 -0400)]
Refactor v1.1 tests and turn them on.
Previously, the "V11" text fixture was empty.
David Neto [Fri, 22 Apr 2016 00:50:11 +0000 (20:50 -0400)]
Add CHANGES file with high level software history
Add a high level version number for SPIRV-Tools, beginning
with v2016.0-dev. The README describes the format of the
version number.
The high level version number is extracted from the CHANGES
file. That works around:
- stale-bait for when we don't add tags to the repository
- our inability to add tags to the repository
Option --version causes spirv-as, spirv-dis, and spirv-val to
show the high level version number.
Add spvSoftwareVersionString to return the C-string for
the high level version number.
Add spvSoftwareVersionDetailsString() so that clients can get
more information if they want to.
Also allows us to clean up the uses in the tool executables files,
so now only one file includes build-version.inc.
Move the update-build-version logic to the only
CMakeLists file that needs it.
The update build version script takes a new argument
to name the output file.
Dejan Mircevski [Fri, 22 Apr 2016 18:56:07 +0000 (14:56 -0400)]
Make grammar-generation fail if over 64 capabilities.
Dejan Mircevski [Fri, 22 Apr 2016 18:05:55 +0000 (14:05 -0400)]
Eliminate kCapabilitiesMin/Max.
Dejan Mircevski [Fri, 22 Apr 2016 17:35:15 +0000 (13:35 -0400)]
Turn module_capabilities_ into a spv_capability_mask_t.
Dejan Mircevski [Thu, 21 Apr 2016 21:16:32 +0000 (17:16 -0400)]
Bump kCapabilitiesMaxValue for v1.1.
Dejan Mircevski [Thu, 21 Apr 2016 21:12:47 +0000 (17:12 -0400)]
Fix Windows build.
Replace `#ifdef 0` with comment inside a macro call.
Lei Zhang [Thu, 21 Apr 2016 19:50:23 +0000 (15:50 -0400)]
Avoid going out of bound in advanceLine() and refine comments.
Dejan Mircevski [Thu, 21 Apr 2016 19:46:08 +0000 (15:46 -0400)]
Add tests for SubgroupDispatch.
Introduced in v1.1, SubgroupDispatch adds the following:
- two new execution modes
- one new capability
- two new opcodes
Extend ValidateBase methods to take a spv_target_env. Replace the
context_ member with ScopedContext inside the said methods. Give
ScopedContext wider visibility by moving it outside
TextToBinaryTestBase.
Dejan Mircevski [Tue, 19 Apr 2016 20:53:47 +0000 (16:53 -0400)]
Mention 1.1 support in the README.
Dejan Mircevski [Tue, 19 Apr 2016 02:25:35 +0000 (22:25 -0400)]
First tests for SPIR-V v1.1.
Add test for named-barrier instructions and capability.
Add spv_target_env as an optional argument to CompileSuccessfully() and
CompileFailure(). Currently defaults to UNIVERSAL_1_0, though that
could change in the future.
Make spv_context a local variable in test methods instead of a
TextToBinaryTestBase member. Introduce ScopedContext to make temp
contexts easier.
Lei Zhang [Mon, 18 Apr 2016 20:48:33 +0000 (16:48 -0400)]
Fix wrong grammar .inc file dependency introduced by copy-paste.
Lei Zhang [Fri, 15 Apr 2016 16:51:55 +0000 (12:51 -0400)]
Rework CMake rules for building grammar tables.
Use add_custom_command() to generate all .inc files and manage
dependency with set_source_files_properties().
Derek Sollenberger [Mon, 18 Apr 2016 14:59:47 +0000 (10:59 -0400)]
Update cmake files to support Android as a build target.
This enables cross-compiling to Android using scripts like
cmake-android that update the system name to reflect the build
target.
David Neto [Mon, 18 Apr 2016 14:32:49 +0000 (10:32 -0400)]
Make directories more safely.
Avoid a race on check-for-dir vs. make-dir
when running the inc-file generation script concurrently.
Dejan Mircevski [Thu, 7 Apr 2016 18:41:34 +0000 (14:41 -0400)]
Add SPIR-V 1.1 grammar.
Switch to SPIR-V 1.1 as default.
Introduce SPV_ENV_UNIVERSAL_1_1.
Add "1_0" to the internal variable names.
Add spv_target_env to spv*GetTable().
Dejan Mircevski [Thu, 7 Apr 2016 18:09:48 +0000 (14:09 -0400)]
Mark SPIR-V grammar files as 1.0.
Factor the CMake generate_grammar_tables invocation into a function
parameterized on the version.
Lei Zhang [Thu, 14 Apr 2016 22:35:14 +0000 (18:35 -0400)]
GLSL grammar update.
* Add names to operands.
* Fix capability requirements for UnpackDouble2x32.
* Fix operands list error for NClamp.
John Kessenich [Thu, 14 Apr 2016 19:09:09 +0000 (13:09 -0600)]
Merge pull request #181 from antiagainst/readme
Use footnotes for links in README.md.
Lei Zhang [Thu, 14 Apr 2016 18:10:35 +0000 (14:10 -0400)]
Use footnotes for links in README.md.