Oopenharmony_ciCVE 513337118
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
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 年前 | |
Move SkSLDefines to src/sksl/ It has no more uses in include/ Change-Id: Iefdf22e5b88625c84c1305a24184503e96561a98 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/753497 Commit-Queue: Kevin Lubick <kjlubick@google.com> Auto-Submit: Kevin Lubick <kjlubick@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: John Stiles <johnstiles@google.com> | 2 年前 | |
Remove SkSL program size limit Bug: 40044195 Change-Id: Ic456211b208421a978fbbda103dda8fead063ab2 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/891757 Reviewed-by: Michael Ludwig <michaelludwig@google.com> Commit-Queue: Brian Osman <brianosman@google.com> | 1 年前 | |
Report an error if variables are scoped incorrectly. Now that symbol table management has been cleaned up, it's a good idea to prevent backsliding. We now have a warning to let us know if variables are not declared inside the closest symbol table. Change-Id: Iadef1f31264e38b86c3f565e08d57cc87698c50b Reviewed-on: https://skia-review.googlesource.com/c/skia/+/802857 Auto-Submit: John Stiles <johnstiles@google.com> Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: Brian Osman <brianosman@google.com> | 2 年前 | |
Replace bug ids with mapped ids in the new issue tracker Bug references of the form: skia:NNNN crbug.com/skia/NNNN https://crbug.com/skia/NNNN http://skbug.com/NNNN skbug.com/NNNN skbug:NNNN are replaced with skbug.com/MMMMMMMM where NNNN maps to MMMMMMMM in the new issue tracker. Change-Id: Ifdc83c39e95e369135e1daa4ebb157977fecb36f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1002016 Auto-Submit: Shahbaz Youssefi <syoussefi@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Brett Somocurcio <brettos@google.com> | 1 年前 | |
Move DSL and most private SkSL headers out of include/. We aren't supporting DSL as public API, so its headers have moved back into src/sksl/. Most SkSL references in include/private/ were in support of DSL, so these have been moved back into src/sksl/ as well. Change-Id: I9b7f965f9e77fb6e24de388d79db82c56b51bad3 Bug: skia:14183 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/671958 Reviewed-by: Kevin Lubick <kjlubick@google.com> Commit-Queue: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> | 3 年前 | |
CVE 513337118 Signed-off-by: whuqincheng <liweiyang1@huawei.com> | 28 天前 | |
Address clang-tidy suggestion Change-Id: Iab51b94ad9eb3bf214bb6adc6cd030c20be6a2a6 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/884517 Auto-Submit: Kaylee Lubick <kjlubick@google.com> Reviewed-by: John Stiles <johnstiles@google.com> | 1 年前 | |
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 年前 | |
Replace bug ids with mapped ids in the new issue tracker Bug references of the form: skia:NNNN crbug.com/skia/NNNN https://crbug.com/skia/NNNN http://skbug.com/NNNN skbug.com/NNNN skbug:NNNN are replaced with skbug.com/MMMMMMMM where NNNN maps to MMMMMMMM in the new issue tracker. Change-Id: Ifdc83c39e95e369135e1daa4ebb157977fecb36f Reviewed-on: https://skia-review.googlesource.com/c/skia/+/1002016 Auto-Submit: Shahbaz Youssefi <syoussefi@google.com> Commit-Queue: Robert Phillips <robertphillips@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com> Reviewed-by: Brett Somocurcio <brettos@google.com> | 1 年前 | |
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 年前 | |
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 年前 | |
Treat non-side-effecting prefix expressions as trivial. Negation and inversion are almost always cheap enough to be cloned without hesitation. Change-Id: I4e1b31c67fc6249aef46746be04b5d9eec46a05d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/773956 Reviewed-by: Michael Ludwig <michaelludwig@google.com> Commit-Queue: Michael Ludwig <michaelludwig@google.com> Auto-Submit: John Stiles <johnstiles@google.com> Commit-Queue: John Stiles <johnstiles@google.com> | 2 年前 | |
Minor fixes for buffet build Certain toolchains compile headers independently, and thus noticed some missing #includes. This also removes the default_malloc_impl module, as that gives linking errors in some environments because a symbol is potentially missing depending on link order. See also http://cl/572301023 Bug: b/40045064 Bug: b/237076898 Change-Id: Ia0a3f32bdab9bc92d09eddd3e5dc7c190cb5caf3 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/765357 Reviewed-by: Leandro Lovisolo <lovisolo@google.com> | 2 年前 | |
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 年前 | |
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 年前 | |
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 年前 | |
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 年前 | |
Reland "Add defensive check for unreachable specialized functions." This reverts commit 56de22c39d0ed5601175f8f4cee72233864be72b. Reason for revert: should work in conjunction with http://review.skia.org/884157 Original change's description: > Revert "Add defensive check for unreachable specialized functions." > > This reverts commit 946ca24d03ab131a25cc71b91c7662777d0f5a58. > > Reason for revert: breaks some tests in SPIR-V > > Original change's description: > > Add defensive check for unreachable specialized functions. > > > > Generally, SkSL will eliminate unreachable functions, so it's > > unlikely that we will ever experience problems here. However, > > in the rare event that an unreachable function sneaks through, > > we will now eliminate it during specialization, instead of > > allowing the backend to emit the function without applying any > > specializations to it (which would be bad, since by definition > > a specialized function contains code which isn't natively > > allowed in the backend for some reason). > > > > In practice, http://review.skia.org/883097 fixed the one case > > that I am aware of which could cause this to occur. (Of course, > > there might be other cases which I haven't discovered yet.) > > > > Bug: b/328051548 > > Change-Id: I44f3fff65e186922654410dedef300c29f159eae > > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/882813 > > Auto-Submit: John Stiles <johnstiles@google.com> > > Commit-Queue: John Stiles <johnstiles@google.com> > > Reviewed-by: Nathan Sanchez <nathanasanchez@google.com> > > Bug: b/328051548 > Change-Id: Ic2f55cee40c69c881c7c6af4c48ed970e5ca6291 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/883178 > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> > Auto-Submit: John Stiles <johnstiles@google.com> > Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bug: b/328051548 Change-Id: Ia8c8ced9712215bdfbcae0bca6c587a50cf83c89 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/884158 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> | 1 年前 | |
Add design doc link for go/sksl-function-specialization. Bug: b/328051548 Change-Id: I1765c6e0f2b346e45ee675770418463937d41dbd Reviewed-on: https://skia-review.googlesource.com/c/skia/+/885196 Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Nathan Sanchez <nathanasanchez@google.com> Commit-Queue: John Stiles <johnstiles@google.com> | 1 年前 | |
Add WGSL support for switches that do not have fallthrough cases. The WGSL code generator now supports switch statements as long as no cases fall through. We mirror WGSL's native abilities and emit a comma-separated list of switch-case values when multiple case X: statements are placed back-to-back. We currently tack on a comment indicating the cases that were not handled due to fallthrough. In a followup CL, the non-native cases will be emulated via if-else statements. This will mimic the rewrite-switches mode in the GLSL code generator, but we will only rewrite the cases that actually violate the WGSL switch rules (either by falling through, or by being fallen-into). Bug: skia:13092 Change-Id: I9c61399a9573a2bc57510fd4bee994f4312894f9 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/710257 Reviewed-by: Brian Osman <brianosman@google.com> Reviewed-by: Arman Uguray <armansito@google.com> | 3 年前 | |
Use a Block to hold the body of a switch statement. Conceptually, the body of a switch statement is a braced block with the restriction that it must contain switch-cases. This simplifies some code that deals with blocks and symbol table scopes. Change-Id: Ib3948d2133a3e5c82cdc67366952fee44f82918d Reviewed-on: https://skia-review.googlesource.com/c/skia/+/801837 Auto-Submit: John Stiles <johnstiles@google.com> Reviewed-by: Brian Osman <brianosman@google.com> Commit-Queue: Brian Osman <brianosman@google.com> | 2 年前 |