mewin/glslang
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
4,086 Commits 1 Branch 0 Tags
Commit Graph

4086 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Daniel Koch
d9b931e2e8 Fix warning in build 
glslang/MachineIndependent/SymbolTable.h:892:41: error: comparison of integers of
different signs: 'int' and 'const uint32_t' (aka 'const unsigned int') [-Werror,-Wsign-compare]
            uint64_t level = currentLevel() > MaxLevelInUniqueID ? MaxLevelInUniqueID : currentLevel();
                             ~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~~~
1 error generated.
 2021-02-19 16:48:50 -05:00
greg-lunarg
e56beaee73
Merge pull request #2536 from greg-lunarg/chg0 
Update CHANGES for 11.1.0
 2021-02-18 15:18:16 -07:00
Greg Fischer
ecd9bb202f Update CHANGES for 11.1.0 2021-02-18 12:04:17 -07:00
greg-lunarg
51b86c1a2b
Merge pull request #2532 from juliusikkala/master 
Pass environment through PreprocessDeferred
 2021-02-15 13:41:48 -07:00
greg-lunarg
b0f8a0c3ab
Merge pull request #2458 from ShchchowAMD/unique_id-fix 
Fix issue for new unique id system.
 2021-02-15 13:22:14 -07:00
Chow
93b400f267 Fix issue for new unique id system. Add level bits to help verifying symbols and split symbol tables. 
For intermediates rebuilding, now need manually amending level bits for redeclaring built-ins.
 2021-02-15 20:35:20 +08:00
Julius Ikkala
6274ec5c20 Pass environment through PreprocessDeferred 2021-02-14 14:24:17 +02:00
greg-lunarg
6ab923c69d
Merge pull request #2531 from greg-lunarg/kg116 
Update spirv-tools and spirv-header known good.
 2021-02-11 17:30:24 -07:00
Greg Fischer
6866807d6a Update spirv-tools and spirv-header known good. 
This is being done to allow Vulkan validation layers to pick up
the new differentiated error code support in spirv-tools for
GPU-AV.
 2021-02-11 15:36:28 -07:00
greg-lunarg
4afd691772
Merge pull request #2526 from proydakov/regression-python3-is-required-even-without-using-SPIRV-Tools 
Fixed issues 2496. Used option 4: parse_version.cmake to avoid python3 usage.
 2021-02-09 10:02:24 -07:00
Evgeny Proydakov
e3dab6f0d2 Code review. Thx ben-clayton for comments. 2021-02-09 12:14:52 +03:00
Evgeny Proydakov
05798c17fb Fixed issues 2496. Used option 4: parse_version.cmake to avoid python3 usage. 
https://github.com/KhronosGroup/glslang/issues/2496
 2021-02-09 12:14:52 +03:00
greg-lunarg
386b4fcb28
Merge pull request #2523 from dneto0/remove-stray-semi 
Remove useless semicolon
 2021-02-03 15:50:43 -07:00
David Neto
57e6a38f2a Remove useless semicolon 
The extra semicolon causes a build failure if warnings are turned up high, and
warnings-as-errors is on. (-Werror=extra-semi)
 2021-02-03 14:14:11 -05:00
greg-lunarg
5c4f421121
Merge pull request #2517 from glebm/fix-ubsan-localintermediate 
callGraph.push_front -> emplace_front to fix UBSAN
 2021-02-01 10:53:37 -07:00
greg-lunarg
6d68720cbb
Merge pull request #2512 from cmarcelo/GL_EXT_shared_memory_block 
Add support for GL_EXT_shared_memory_block
 2021-02-01 09:29:52 -07:00
Gleb Mazovetskiy
546f626c58 callGraph.push_front -> emplace_front to fix UBSAN 
UBSAN rightly complains on `push_front` here:

    glslang/MachineIndependent/localintermediate.h💯8: runtime error: load of value 160, which is not a valid value for type 'bool'
    #0 in glslang::TCall::TCall(glslang::TCall&&) glslang/MachineIndependent/localintermediate.h:100
    #1 in void __gnu_cxx::new_allocator<std::_List_node<glslang::TCall> >::construct<glslang::TCall, glslang::TCall>(glslang::TCall*, glslang::TCall&&) /usr/include/c++/10/ext/new_allocator.h:150
    #2 in void std::allocator_traits<std::allocator<std::_List_node<glslang::TCall> > >::construct<glslang::TCall, glslang::TCall>(std::allocator<std::_List_node<glslang::TCall> >&, glslang::TCall*, glslang::TCall&&) /usr/include/c++/10/bits/alloc_traits.h:512
    #3 in std::_List_node<glslang::TCall>* std::__cxx11::list<glslang::TCall, std::allocator<glslang::TCall> >::_M_create_node<glslang::TCall>(glslang::TCall&&) (...)
    #4 in void std::__cxx11::list<glslang::TCall, std::allocator<glslang::TCall> >::_M_insert<glslang::TCall>(std::_List_iterator<glslang::TCall>, glslang::TCall&&) /usr/include/c++/10/bits/stl_list.h:1911
    #5 in std::__cxx11::list<glslang::TCall, std::allocator<glslang::TCall> >::push_front(glslang::TCall&&) /usr/include/c++/10/bits/stl_list.h:1167
    #6 in glslang::TIntermediate::addToCallGraph(TInfoSink&, std::__cxx11::basic_string<char, std::char_traits<char>, glslang::pool_allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, glslang::pool_allocator<char> > const&) glslang/MachineIndependent/Intermediate.cpp:2860

What happens here:

1. TCall's bool fields are not initialized on construction.
2. `push_front` move the `TCall` passed into it.
3. The move constructor copies unitialized bool, which may have an
   out-of-range value.

What this fix does:

Calls `emplace_back` to ensure no copy/move constructor is called.

Fixes #2222
Refs #2112
 2021-01-30 16:06:08 +00:00
Caio Marcelo de Oliveira Filho
4bfbf62794 Add support for GL_EXT_shared_memory_block 
Uses SPV_KHR_workgroup_memory_explicit_layout.  Note that if
GL_EXT_scalar_block_layout is enabled, Workgroup blocks can also use
scalar layout.
 2021-01-29 11:23:05 -08:00
Caio Marcelo de Oliveira Filho
3785ddd59a Update known_good.json to pick up SPV_KHR_workgroup_memory_explicit_layout 
Also update the test expectations regarding validation accordingly and
the in-tree spirv.hpp copy.
 2021-01-29 11:22:22 -08:00
Caio Marcelo de Oliveira Filho
a38df83d3e Consider GL_EXT_scalar_block_layout when validating SPIR-V 
If GL_EXT_scalar_block_layout is requested by the shader, set the
option to allow scalar blocks in the SPIR-V validator.

Fix the existing tests using scalar layout to not expect "Validation
failed".

Fixes #2400.
 2021-01-28 14:11:24 -08:00
greg-lunarg
f7c43377f0
Merge pull request #2513 from jeffbolznv/missingcap 
Add missing capability when QueueFamily scope is used
 2021-01-28 10:40:45 -07:00
Jeff Bolz
bfd84a39f2 Add missing capability when QueueFamily scope is used 
Also, if this capability is added and the memory model is not
otherwise enabled by pragma, enable it as part of postprocessing.
 2021-01-27 13:14:34 -06:00
greg-lunarg
36df92e4a0
Merge pull request #2511 from alan-baker/GL_EXT_null_initializer 
Implement GL_EXT_null_initializer
 2021-01-26 12:05:11 -07:00
John Kessenich
c739e03748 Implement GL_EXT_null_initializer 
Adds null initializer syntax (empty braces)
Allows null initialization of shared variables
 2021-01-25 15:54:52 -05:00
greg-lunarg
6abdde3ce5
Merge pull request #2508 from corporateshark/master 
Fix missing enum values in the C-interface #2507
 2021-01-21 10:58:14 -07:00
Sergey Kosarevsky
a720028cdd Fix missing enum values in the C-interface #2507 2021-01-21 02:33:16 +03:00
greg-lunarg
9b4e82efb4
Merge pull request #2505 from ben-clayton/update-licnese-checker-cfg 
license-checker.cfg: Update rules
 2021-01-19 11:05:02 -07:00
greg-lunarg
5b7d68ca6b
Merge pull request #2504 from cdotstout/fuchsia-cmake 
Add CMake support for Fuchsia
 2021-01-19 10:59:05 -07:00
Ben Clayton
752028582f license-checker.cfg: Update rules 
The license classification rules in github.com/google/licensecheck have been updated.
 2021-01-18 12:27:00 +00:00
Craig Stout
17a83a9b33 Add CMake support for Fuchsia 2021-01-15 10:30:23 -08:00
greg-lunarg
3de5cfe50e
Merge pull request #2498 from dezinjob/fix-option-replacement-vs 
Fix option replacement when flags are empty
 2021-01-06 19:29:37 -07:00
greg-lunarg
480f6039e0
Merge pull request #2500 from jeremy-lunarg/hayes-2497 
Prevent HLSL input under OpenGL semantics
 2021-01-06 10:48:00 -07:00
greg-lunarg
f77085c307
Merge pull request #2499 from greg-lunarg/i2494 
Fix debugInfo test to target vulkan1.1 as intended
 2021-01-05 18:38:50 -07:00
Jeremy Hayes
b73afa8631 Prevent HLSL input under OpenGL semantics 
Fix #2497.
 2021-01-05 15:55:58 -07:00
Greg Fischer
e453088a8a Fix debugInfo test to target vulkan1.1 as intended 
Fixes #2494
 2021-01-05 14:45:11 -07:00
Denis Zalevskiy
78dbd7514a Fix option replacement when flags are empty 
For the scenario when cmake is executed from MSVC with empty CXX flags.

Signed-off-by: Denis Zalevskiy <denis.zalevskiy@basemark.com>
 2021-01-05 15:07:24 +02:00
greg-lunarg
9325cc013e
Merge pull request #2393 from nihui/patch-3 
Fix build with android ndk r16b
 2020-12-24 12:53:39 -07:00
nihuini
08e4874a56 Merge remote-tracking branch 'upstream/master' into patch-3 2020-12-24 10:33:22 +08:00
nihui
5c202faffb
Update ParseHelper.cpp 2020-12-23 21:56:23 +08:00
greg-lunarg
f426f65229
Merge pull request #2490 from greg-lunarg/i2476 
Fix tess factor cut and paste error
 2020-12-18 10:19:52 -07:00
Greg Fischer
efe4586fad Fix cut and paste error 
Fixes #2476
 2020-12-17 22:57:50 -07:00
greg-lunarg
6d5916d4a9
Merge pull request #2451 from rdb/dx9-round-semantics 
Don't compile round() to roundEven in DX9 compatibility mode
 2020-12-16 13:55:00 -07:00
rdb
589aaff11c Don't use roundEven() to implement round() in DX9 compatibility mode 2020-12-16 18:35:42 +01:00
John Kessenich
a5be11bd8b
Merge pull request #2485 from proydakov/fixed-hlsl-off-compile-warning 
Fixed compile warning in reflection.cpp for ENABLE_HLSL = 0 build. [-Wunused-parameter]
 2020-12-15 12:16:17 -07:00
John Kessenich
b6fd41df69
Merge pull request #2487 from dneto0/validate-imageread-result 
Update SPIRV-Tools known-good: validate OpImageRead result type
 2020-12-15 12:15:32 -07:00
David Neto
1ef5e20290 Test updates for ImageRead result type validation 
See #2486
 2020-12-15 12:19:00 -05:00
David Neto
3805671ce8 Update known_good, pick up ImageRead result validation 
Pick up https://github.com/KhronosGroup/SPIRV-Tools/pull/4072
which validates the result type of OpImageRead.

This also requires test expectation updates.

See #2486 tracks the needed upate to Glslang code generation.
 2020-12-15 12:09:50 -05:00
Evgeny Proydakov
dc8b1d0264 Fixed compile warning in reflection.cpp for ENABLE_HLSL = 0 build. [-Wunused-parameter] 2020-12-15 18:05:32 +03:00
Daniel Koch
c0bcfaf3ba
Fix SPV return type of rayQueryGetIntersectionInstanceShaderBindingTableRecordOffsetEXT (#2484) 
Issue #2483

According to GLSL spec the prototype is:
        uint rayQueryGetIntersectionInstanceShaderBindingTableRecordOffsetEXT(rayQueryEXT q, bool committed);

but that was incorrectly getting translated to SPIRV as an `int`, and this was
causing SPIR-V validation errors when used.

Added explicit testing for the return types of all the builtin functions in GL_EXT_ray_query
 2020-12-12 10:34:24 -07:00
greg-lunarg
ab66a91d62
Merge pull request #2482 from greg-lunarg/gtfix3 
Update README to avoid googletest breakage
 2020-12-11 15:08:50 -07:00