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

Upgrade glslang 11.16.0+ #3

Closed

Conversation

jsmall-zzz
Copy link

@jsmall-zzz jsmall-zzz commented Oct 5, 2021

Upgrades to glslang 11.16.0, plus some subsequent PRs up to KhronosGroup#2768

kevin-mccullough and others added 30 commits July 9, 2021 09:51
…er or shared variable.

Signed-off-by: ZhiqianXia <xzq0528@outlook.com>
…der5.

Signed-off-by: ZhiqianXia <xzq0528@outlook.com>
Fix per-set argument parsing in standalone
Fix some instances of -Wunused-but-set-variable.
Fix OpImageRead result type when compiling HLSL

- Per the Vulkan spec, OpImageRead must return a 4-component vector always. When compiling HLSL, loads from a RWTexture of a template type with < 4 components would incorrectly generate an OpImageRead with a < 4 component result, resulting in validation errors.
- This was previously fixed for OpImageFetch in commit 4425f24; this commit does the same thing for OpImageRead.
- Added associated tests and expanded existing image fetch tests to check all the different types of textures, in both float and int incarnations, for completeness.
- Update other HLSL tests involving OpImageRead
Full vector and fully specified vector swizzle stores are not affected by this change, only partial swizzles ie swizzles with fewer components than the vector being stored to.

Previously the vector being stored to loaded and any components not specified in the swizzle were used to create a full store to the vector.

While this change generates more SPIR-V instructions, it is necessary for correctness.

Fixes KhronosGroup#2518.
Generate separate stores for partially swizzled memory stores
Support Uint to Int implicit conversions at #extension GL_ARB_gpu_shader5.
Atomic memory function can only be used for shader storage block member or shared variable.
…ntime_override_optional

add possibility to not override runtime of MSVC
Allow layout(std430) uniform with GL_EXT_scalar_block_layout
Add support for pragma STDGL invariant(all)
Checked REGEX REPLACE for consistency failure
Seen with += and ++ ops on swizzled lvalues. Was using stale access chain.

Fixes 2725.
Fix seperate stores to swizzled lvalue.
Add support for GL_NV_ray_tracing_motion_blur.
greg-lunarg and others added 25 commits August 18, 2021 12:59
Support the atomic_counter_xxxARB  built-in functons at #extension GL_ARB_shader_atomic_counter_ops.
Update known goods and CHANGES for 11.6.0
Fix -Wsign-compare warnings.
Fix -Wunused-parameter warnings.
Perform update_precision on constructors and converts
MINGW_HAS_SECURE_API can be defined as 0, but this will be ignored here without this change.
Without these *_s "safe" functions, this code will also build on Windows XP.
Run update_precision() on array and matrix constructors.
…re/master

Change MINGW_HAS_SECURE_API checks.
When using GL_EXT_vulkan_glsl_relaxed, check to see if a
uniform already exists in the block uniform block, and if so,
ensure they are the same type. Otherwise, avoid growing the block
with a duplicate.
instead of allowing for multiple declarations of the variable in the
resulting SPIR-V, instead use a retargeted mechanism to cause references
to gl_VertexID and gl_InstanceID to use the gl_VertexIndex and
gl_InstanceIndex symbol.
…licateUniform

GL_EXT_vulkan_glsl_relaxed - avoid growing the global uniform block with duplicates
GL_EXT_vulkan_glsl_relaxed - retarget gl_VertexID to gl_VertexIndex
…izeArrayTesselationStageHandling

Fix isIoResizeArray() tessellation stage handling
* add ability to upgrade uniform block to push constants assuming it fits within the size limit imposed by the caller
* allow selecting the packing for the auto push constants
* check the size using the potential layout packing of the push constants
Fix unreachable code in getSampledType()
@jsmall-zzz jsmall-zzz changed the title Upgrade glslang 0.16.0+ Upgrade glslang 11.16.0+ Oct 5, 2021
@jsmall-zzz jsmall-zzz closed this Oct 5, 2021
@jsmall-zzz jsmall-zzz deleted the feature/glslang-0.16.0+ branch June 29, 2022 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet