[Impeller] Start stroke tessellation in compute (#39543)

[Impeller] Start stroke tessellation in compute

Author: dnfield

ci/licenses_golden/excluded_files

@@ -138,6 +138,7 @@
 ../../../flutter/impeller/geometry/geometry_unittests.h
 ../../../flutter/impeller/image/README.md
 ../../../flutter/impeller/playground
+../../../flutter/impeller/renderer/compute_subgroup_unittests.cc
 ../../../flutter/impeller/renderer/compute_unittests.cc
 ../../../flutter/impeller/renderer/device_buffer_unittests.cc
 ../../../flutter/impeller/renderer/host_buffer_unittests.cc

ci/licenses_golden/licenses_flutter

@@ -1088,6 +1088,7 @@ ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/color.glsl + ../.
 ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/constants.glsl + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/gaussian.glsl + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/gradient.glsl + ../../../flutter/LICENSE
+ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/path.glsl + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/texture.glsl + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/transform.glsl + ../../../flutter/LICENSE
 ORIGIN: ../../../flutter/impeller/compiler/shader_lib/impeller/types.glsl + ../../../flutter/LICENSE
@@ -3584,6 +3585,7 @@ FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/color.glsl
 FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/constants.glsl
 FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/gaussian.glsl
 FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/gradient.glsl
+FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/path.glsl
 FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/texture.glsl
 FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/transform.glsl
 FILE: ../../../flutter/impeller/compiler/shader_lib/impeller/types.glsl

impeller/compiler/compiler.cc

@@ -7,7 +7,9 @@
 #include <array>
 #include <filesystem>
 #include <memory>
+#include <optional>
 #include <sstream>
+#include <string>
 #include <utility>
 
 #include "flutter/fml/paths.h"
@@ -25,16 +27,36 @@ const uint32_t kFragBindingBase = 128;
 const size_t kNumUniformKinds =
     static_cast<int>(shaderc_uniform_kind::shaderc_uniform_kind_buffer) + 1;
 
+static uint32_t ParseMSLVersion(const std::string& msl_version) {
+  std::stringstream sstream(msl_version);
+  std::string version_part;
+  uint32_t major = 1;
+  uint32_t minor = 2;
+  uint32_t patch = 0;
+  if (std::getline(sstream, version_part, '.')) {
+    major = std::stoi(version_part);
+    if (std::getline(sstream, version_part, '.')) {
+      minor = std::stoi(version_part);
+      if (std::getline(sstream, version_part, '.')) {
+        patch = std::stoi(version_part);
+      }
+    }
+  }
+  if (major < 1 || minor < 2) {
+    std::cerr << "--metal-version version must be at least 1.2. Have "
+              << msl_version << std::endl;
+  }
+  return spirv_cross::CompilerMSL::Options::make_msl_version(major, minor,
+                                                             patch);
+}
+
 static CompilerBackend CreateMSLCompiler(const spirv_cross::ParsedIR& ir,
                                          const SourceOptions& source_options) {
   auto sl_compiler = std::make_shared<spirv_cross::CompilerMSL>(ir);
   spirv_cross::CompilerMSL::Options sl_options;
   sl_options.platform =
       TargetPlatformToMSLPlatform(source_options.target_platform);
-  // If this version specification changes, the GN rules that process the
-  // Metal to AIR must be updated as well.
-  sl_options.msl_version =
-      spirv_cross::CompilerMSL::Options::make_msl_version(1, 2);
+  sl_options.msl_version = ParseMSLVersion(source_options.metal_version);
   sl_options.use_framebuffer_fetch_subpasses = true;
   sl_compiler->set_msl_options(sl_options);
 
@@ -357,9 +379,9 @@ Compiler::Compiler(const fml::Mapping& source_mapping,
           shaderc_optimization_level::shaderc_optimization_level_performance);
       spirv_options.SetTargetEnvironment(
           shaderc_target_env::shaderc_target_env_vulkan,
-          shaderc_env_version::shaderc_env_version_vulkan_1_0);
+          shaderc_env_version::shaderc_env_version_vulkan_1_1);
       spirv_options.SetTargetSpirv(
-          shaderc_spirv_version::shaderc_spirv_version_1_0);
+          shaderc_spirv_version::shaderc_spirv_version_1_3);
       break;
     case TargetPlatform::kRuntimeStageMetal:
     case TargetPlatform::kRuntimeStageGLES:
@@ -437,7 +459,11 @@ Compiler::Compiler(const fml::Mapping& source_mapping,
                    << ShaderCErrorToString(spv_result_->GetCompilationStatus())
                    << ". " << spv_result_->GetNumErrors() << " error(s) and "
                    << spv_result_->GetNumWarnings() << " warning(s).";
-    if (spv_result_->GetNumErrors() > 0 || spv_result_->GetNumWarnings() > 0) {
+    // It should normally be enough to check that there are errors or warnings,
+    // but some cases result in no errors or warnings and still have an error
+    // message. If there's a message we should print it.
+    if (spv_result_->GetNumErrors() > 0 || spv_result_->GetNumWarnings() > 0 ||
+        !spv_result_->GetErrorMessage().empty()) {
       COMPILER_ERROR_NO_PREFIX << spv_result_->GetErrorMessage();
     }
     return;

impeller/compiler/impellerc_main.cc

@@ -74,6 +74,7 @@ bool Main(const fml::CommandLine& command_line) {
       switches.entry_point);
   options.json_format = switches.json_format;
   options.gles_language_version = switches.gles_language_version;
+  options.metal_version = switches.metal_version;
 
   Reflector::Options reflector_options;
   reflector_options.target_platform = switches.target_platform;

impeller/compiler/shader_lib/impeller/BUILD.gn

@@ -10,6 +10,7 @@ copy("impeller") {
     "constants.glsl",
     "gaussian.glsl",
     "gradient.glsl",
+    "path.glsl",
     "texture.glsl",
     "transform.glsl",
     "types.glsl",

impeller/compiler/shader_lib/impeller/path.glsl

@@ -0,0 +1,68 @@
+// Copyright 2013 The Flutter Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef PATH_GLSL_
+#define PATH_GLSL_
+
+#define MOVE 0
+#define LINE 1
+#define QUAD 2
+#define CUBIC 3
+
+struct LineData {
+  vec2 p1;
+  vec2 p2;
+};
+
+struct QuadData {
+  vec2 p1;
+  vec2 cp;
+  vec2 p2;
+};
+
+struct CubicData {
+  vec2 p1;
+  vec2 cp1;
+  vec2 cp2;
+  vec2 p2;
+};
+
+struct Position {
+  uint index;
+  uint count;
+};
+
+/// Solve for point on a quadratic Bezier curve defined by starting point `p1`,
+/// control point `cp`, and end point `p2` at time `t`.
+vec2 QuadraticSolve(QuadData quad, float t) {
+  return (1.0 - t) * (1.0 - t) * quad.p1 +  //
+         2.0 * (1.0 - t) * t * quad.cp +    //
+         t * t * quad.p2;
+}
+
+vec2 CubicSolve(CubicData cubic, float t) {
+  return (1. - t) * (1. - t) * (1. - t) * cubic.p1 +  //
+         3 * (1. - t) * (1. - t) * t * cubic.cp1 +    //
+         3 * (1. - t) * t * t * cubic.cp2 +           //
+         t * t * t * cubic.p2;
+}
+
+/// Used to approximate quadratic curves using parabola.
+///
+/// See
+/// https://raphlinus.github.io/graphics/curves/2019/12/23/flatten-quadbez.html
+float ApproximateParabolaIntegral(float x) {
+  float d = 0.67;
+  return x / (1.0 - d + sqrt(sqrt(pow(d, 4) + 0.25 * x * x)));
+}
+
+bool isfinite(float f) {
+  return !isnan(f) && !isinf(f);
+}
+
+float Cross(vec2 p1, vec2 p2) {
+  return p1.x * p2.y - p1.y * p2.x;
+}
+
+#endif

impeller/compiler/source_options.h

@@ -27,6 +27,7 @@ struct SourceOptions {
   uint32_t gles_language_version = 100;
   std::vector<std::string> defines;
   bool json_format = false;
+  std::string metal_version;
 
   SourceOptions();
 

impeller/compiler/switches.cc

@@ -131,6 +131,8 @@ Switches::Switches(const fml::CommandLine& command_line)
       gles_language_version(
           stoi(command_line.GetOptionValueWithDefault("gles-language-version",
                                                       "0"))),
+      metal_version(
+          command_line.GetOptionValueWithDefault("metal-version", "1.2")),
       entry_point(
           command_line.GetOptionValueWithDefault("entry-point", "main")) {
   auto language =

impeller/compiler/switches.h

@@ -34,6 +34,7 @@ struct Switches {
   bool json_format;
   SourceLanguage source_language = SourceLanguage::kUnknown;
   uint32_t gles_language_version;
+  std::string metal_version;
   std::string entry_point;
 
   Switches();

impeller/fixtures/BUILD.gn

@@ -5,6 +5,22 @@
 import("//flutter/impeller/tools/impeller.gni")
 import("//flutter/testing/testing.gni")
 
+if (impeller_enable_vulkan || impeller_enable_metal) {
+  impeller_shaders("shader_subgroup_fixtures") {
+    enable_opengles = false
+    name = "subgroup_fixtures"
+
+    # Only need 2.1 for desktop. Running on iOS would require 2.4
+    metal_version = "2.1"
+
+    shaders = [
+      "cubic_to_quads.comp",
+      "quad_polyline.comp",
+      "stroke.comp",
+    ]
+  }
+}
+
 impeller_shaders("shader_fixtures") {
   name = "fixtures"
 
@@ -70,10 +86,12 @@ test_fixtures("file_fixtures") {
     "bay_bridge.jpg",
     "blue_noise.png",
     "boston.jpg",
+    "cubic_to_quads.comp",
     "embarcadero.jpg",
     "flutter_logo_baked.glb",
     "kalimba.jpg",
     "multiple_stages.hlsl",
+    "quad_polyline.comp",
     "resources_limit.vert",
     "sample.comp",
     "sample.frag",
@@ -85,6 +103,7 @@ test_fixtures("file_fixtures") {
     "sa%m#ple.vert",
     "stage1.comp",
     "stage2.comp",
+    "stroke.comp",
     "struct_def_bug.vert",
     "table_mountain_nx.png",
     "table_mountain_ny.png",
@@ -117,4 +136,8 @@ group("fixtures") {
     ":scene_fixtures",
     ":shader_fixtures",
   ]
+
+  if (impeller_enable_vulkan || impeller_enable_metal) {
+    public_deps += [ ":shader_subgroup_fixtures" ]
+  }
 }