commit 6011bbcf48809f72e03d5815a048a963ed927ddf
author John Stiles <johnstiles@google.com> Wed May 19 22:56:18 2021 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu May 20 18:08:35 2021 +0000
tree a1f3d782ab8c17980d383c87292b3a81f4e9503b
parent 2758a3189ade49d228e76963d842ade8e83b24ee

Fix comma operator support with matrices in Metal.

MetalCodeGen would incorrectly identify `(someMatrix, someScalar)` as a
math operation between `someMatrix` and `someScalar` and attempt to
convert `someScalar` to a matrix. If `someScalar` was a Boolean type,
this would lead to an assertion.

The binary expression is now checked more thoroughly before converting
the scalar into a matrix.

Change-Id: Id7e104d5533d8c43375927d4815b83e1a3c36be1
Bug: skia:11125
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/410682
Auto-Submit: John Stiles <johnstiles@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>