tree 96db0caa56f21e07581a64421458e55ed1e9c714
parent 274f07091e0be543cc00a5b2a6848e714aa5c7f8
author Olli Etuaho <oetuaho@nvidia.com> 1431958056 +0300
committer Olli Etuaho <oetuaho@nvidia.com> 1432020984 +0000

Set mUsesDiscardRewriting more conservatively

This fixes a regression from recent refactoring of unfolding short-circuiting
operators. mUsesDiscardRewriting used to be set based on detecting patterns
that the AST traverser now overwrites before the code reaches OutputHLSL. With
this patch mUsesDiscardRewriting is set more conservatively, which ensures that
using discard works correctly also in case the input shader contains the same
kind of pattern as the short-circuiting operator unfolding creates.

TEST=WebGL conformance tests on D3D9
BUG=chromium:488552

Change-Id: Iea0103822a268079593b0913f0b6fc54c745c6e3
Reviewed-on: https://chromium-review.googlesource.com/271761
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Tested-by: Olli Etuaho <oetuaho@nvidia.com>