kc3-lang/angle/src/libANGLE/renderer/wgpu/ShaderWgpu.cpp

• Show log

  Commit

• Hash : 4397ff2f
  Author :
  Date : 2024-10-25T16:55:30
  

  Metal: SeparateCompoundStructDeclarations fails validation
  
  Consider GLSL:
    struct S { int i; } s;
    s=s;
  
  SeparateCompoundStructDeclarations would rewrite this to:
    struct S { int i; };
    S s';
    s=s;
  
  The interm rewrite would rewrite the specification and declaration of s,
  but not the use sites. The use sites would use the old type, and thus
  something that was not in the tree anymore. This would fail the
  validation.
  
  This kind of bug was previously fixed for SeparateDeclarations
  in commit 18fa02bebf901dd8501de3176f6052ae4ce984be.
  
  Fix by adding the logic to SeparateDeclarations, as it is already
  doing almost the exact task, separating `struct S { ..} a, b`.
  
  The separation is tested in GLSLTests.StructInShader and various
  other draw tests. These pass with MSL, but these would also fail
  validation if that was enabled.
  
  Bug: angleproject:375523825
  Change-Id: I1697103d0ba47616dbd3159f36f9e71cb2831c4b
  Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5964899
  Auto-Submit: Kimmo Kinnunen <kkinnunen@apple.com>
  Commit-Queue: Kimmo Kinnunen <kkinnunen@apple.com>
  Reviewed-by: Geoff Lang <geofflang@chromium.org>
  

Download

• src/libANGLE/renderer/wgpu/ShaderWgpu.cpp

• //
  // Copyright 2024 The ANGLE Project Authors. All rights reserved.
  // Use of this source code is governed by a BSD-style license that can be
  // found in the LICENSE file.
  //
  // ShaderWgpu.cpp:
  //    Implements the class methods for ShaderWgpu.
  //
  
  #include "libANGLE/renderer/wgpu/ShaderWgpu.h"
  
  #include "common/debug.h"
  #include "libANGLE/Context.h"
  #include "libANGLE/renderer/ContextImpl.h"
  #include "libANGLE/trace.h"
  
  namespace rx
  {
  
  namespace
  {
  class ShaderTranslateTaskWgpu final : public ShaderTranslateTask
  {
      bool translate(ShHandle compiler,
                     const ShCompileOptions &options,
                     const std::string &source) override
      {
          ANGLE_TRACE_EVENT1("gpu.angle", "ShaderTranslateTaskWgpu::translate", "source", source);
  
          const char *srcStrings[] = {source.c_str()};
          return sh::Compile(compiler, srcStrings, ArraySize(srcStrings), options);
      }
  
      void postTranslate(ShHandle compiler, const gl::CompiledShaderState &compiledState) override {}
  };
  }  // namespace
  
  ShaderWgpu::ShaderWgpu(const gl::ShaderState &data) : ShaderImpl(data) {}
  
  ShaderWgpu::~ShaderWgpu() {}
  
  std::shared_ptr<ShaderTranslateTask> ShaderWgpu::compile(const gl::Context *context,
                                                           ShCompileOptions *options)
  {
      const gl::Extensions &extensions = context->getImplementation()->getExtensions();
      if (extensions.shaderPixelLocalStorageANGLE)
      {
          options->pls = context->getImplementation()->getNativePixelLocalStorageOptions();
      }
  
      options->validateAST = true;
  
      options->separateCompoundStructDeclarations = true;
  
      return std::shared_ptr<ShaderTranslateTask>(new ShaderTranslateTaskWgpu);
  }
  
  std::shared_ptr<ShaderTranslateTask> ShaderWgpu::load(const gl::Context *context,
                                                        gl::BinaryInputStream *stream)
  {
      UNREACHABLE();
      return std::shared_ptr<ShaderTranslateTask>(new ShaderTranslateTask);
  }
  
  std::string ShaderWgpu::getDebugInfo() const
  {
      return "";
  }
  
  }  // namespace rx
  

Download