Star3
55
代码
代码
Issues33
Pull Requests43
流水线
讨论
Wiki
分析
Star3
55
SSkCQDelete all filegroups that supported legacy (non-buffet) Bazel build
d06dc084创建于 2024年8月22日历史提交
文件最后提交记录最后更新时间
BUILD.bazel
Delete all filegroups that supported legacy (non-buffet) Bazel build We don't need most of the flags and all these file groups which had made a big "ball o mud" build. Change-Id: I690f38c3555e390b7c8d459b4649a270c3e03ce2 Bug: b/40045064 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/889266 Commit-Queue: Kaylee Lubick <kjlubick@google.com> Reviewed-by: Ben Wagner <bungeman@google.com>
1 年前
SkSLAddConstToVarModifiers.cpp
Clean up Modifiers, DSLModifiers and ModifierFlags. ModifierFlags have been moved out to a dedicated separate header. Modifiers replaces the old DSLModifiers; it's a simple class holding a position, a layout, and modifier flags. It's used by the Parser and most Convert functions to pass modifiers around. DSLModifiers is now redundant and has been removed. Change-Id: Ib64893fad182b72656eed3da8cc78ce18e64856c Bug: b/40045537 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/732857 Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Arman Uguray <armansito@google.com> Commit-Queue: Arman Uguray <armansito@google.com> Commit-Queue: John Stiles <johnstiles@google.com>
2 年前
SkSLEliminateDeadFunctions.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLEliminateDeadGlobalVariables.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLEliminateDeadLocalVariables.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLEliminateEmptyStatements.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLEliminateUnnecessaryBraces.cpp
Use swizzles to save space during code minification. We can replace half2(foo) with foo.xx, or even float4(5) with 5..xxxx. These swizzles will be converted back into constructors when the minified code is loaded. Change-Id: If2534eadebeab84cc4db4a6e3a2337da2adaeeba Reviewed-on: https://skia-review.googlesource.com/c/skia/+/881480 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLEliminateUnreachableCode.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLFindAndDeclareBuiltinFunctions.cpp
Implement Transform::FindAndDeclareBuiltinStructs. This pass ensures that module structs are included in a program's shared elements, and fixes the Metal and WGSL unit test output from the previous CL. Bug: b/320462028 Change-Id: I16b39ea7f1918ae1f10cd3d96aa0aaf43a228461 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/805776 Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com>
2 年前
SkSLFindAndDeclareBuiltinStructs.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLFindAndDeclareBuiltinVariables.cpp
Update RTFlip handling to avoid caps-bit checks in the front-end. Previously, we had a boolean in the Program::Interface which meant "this program needs RTFlip," and the caps bit for CanUseFragCoord influenced the final state of the boolean. Now, we have a bitfield in the Program::Interface which enumerates the things which can influence the existence of RTFlip (frag coords, clockwise, and derivatives). In the calling code, if any bits are set, we know we need RTFlip; however, we can ignore the RTFlag_FragCoord bit if CanUseFragCoords is disabled. Note that CanUseFragCoords is only disabled in GL, so this logic only exists there. Bug: b/316916102 Change-Id: Ic721d31682e6b3ff44fdcdfdaf9fc29b3dea1da3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/792036 Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
2 年前
SkSLHoistSwitchVarDeclarationsAtTopLevel.cpp
Fix fuzzer-discovered issue with var-decls in a static switch. The fuzzer managed to construct a scenario that required switch variable-hoisting, but the hoisting logic wasn't reached because the static-switch optimization path did not support hoisting. Switch var-decl hoisting now occurs at the end of SwitchStatement::Convert instead of at the end of Make. This fixes the bug; it's also beneficial because it means we no longer re-run the hoisting logic whenever a switch is inlined. Bug: oss-fuzz:70589 Change-Id: I51a12f5b74387d1fa04215bb02afaf547d0e1c70 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/880638 Commit-Queue: John Stiles <johnstiles@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
1 年前
SkSLProgramWriter.h
Remove mentions of -Wweak-template-vtables the Clang warning is being removed, see bug. Bug: chromium:1264351, chromium:1294786 Change-Id: I206f1c891f722cd906fd357176c9f68778e0861f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/464616 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com>
4 年前
SkSLRenamePrivateSymbols.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLReplaceConstVarsWithLiterals.cpp
Migrate SkSL::Module into dedicated header. This is really an overdue change. Previously Module lived in SkSLCompiler.h, and it turns out that many places which #included Compiler were only doing so to access the Module structure. Modules also now hold onto their ModuleType; nothing relies on it today, but it seemed like a reasonable way to spend 1 byte. Change-Id: If52dcd309cbc2d3b9bf7743d108618753c75bf7f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/879603 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前
SkSLReplaceSplatCastsWithSwizzles.cpp
Add push_back_n and pop_back_n to FixedArray. These mirror the TArray equivalents. We didn't have unit testing for these methods previously, so I added some very basic checks. Change-Id: I154ab2b0d3733ab8c96b5d40d3ea802bb4f16af8 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/881481 Reviewed-by: Ben Wagner <bungeman@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Commit-Queue: John Stiles <johnstiles@google.com>
1 年前
SkSLRewriteIndexedSwizzle.cpp
Add a FixedArray class to shrink SkSL::ComponentArray. Using a fixed-size array here saves about 2K of code size, and uses about 16K less RAM when loading the Graphite modules. The FixedArray class was designed to be a drop-in replacement for STArray (at least, enough to pass the basic tests in TArrayTest). For now, it only supports trivial types and up to 255 elements, but these restrictions are fixable if a need arises. Change-Id: I09358427a144af24deb2031813f95759f48efd1f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/844437 Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: John Stiles <johnstiles@google.com>
2 年前
SkSLTransform.cpp
Enable IWYU on SkSLTransform.h. This should fix G3 roll errors. Change-Id: I650665bd21d78119fceea67731978e9be624e359 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/881036 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com>
1 年前
SkSLTransform.h
Use swizzles to save space during code minification. We can replace half2(foo) with foo.xx, or even float4(5) with 5..xxxx. These swizzles will be converted back into constructors when the minified code is loaded. Change-Id: If2534eadebeab84cc4db4a6e3a2337da2adaeeba Reviewed-on: https://skia-review.googlesource.com/c/skia/+/881480 Reviewed-by: Robert Phillips <robertphillips@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Michael Ludwig <michaelludwig@google.com> Auto-Submit: John Stiles <johnstiles@google.com>
1 年前