| commit | 58134e14082d612873fb342e3779666be3b6e544 | [log] [tgz] |
|---|---|---|
| author | Brian Osman <brianosman@google.com> | Thu May 13 14:51:07 2021 -0400 |
| committer | Skia Commit-Bot <skia-commit-bot@chromium.org> | Thu May 13 21:11:10 2021 +0000 |
| tree | 3e7e02c40aef615e2626eed4d61952dfd798f6e2 | |
| parent | 0c2ee32f4c0ed86ad2b1b72b63a5fcda357c0a79 [diff] |
Fix const globals in Metal We were emitting this at global scope (not in Globals). That would lead to errors about the variable needing to be in the constant address space. (You can see the result in ConstArray.metal - the old code was invalid). Also, we were already making references use _globals, so the code was double-wrong (or half-right, depending on your perspective). After the core change, writeVarDeclaration was only used for local scope, and writeModifiers never used the 'globalContext' parameter. The removal of finishLine() changed every test output, unfortunately. Change-Id: Icc1356ba2cc3c339b2f5759b3d18523fd39395bc Reviewed-on: https://skia-review.googlesource.com/c/skia/+/408356 Commit-Queue: Brian Osman <brianosman@google.com> Reviewed-by: John Stiles <johnstiles@google.com>