tree b4d4f9c0a11b1ce5f9caa7492fb8929fe97f0337
parent 7c296ec5848225fbf7c5ab777596d6ad941be6c5
author Nicolas Capens <capn@google.com> 1613679820 -0500
committer Nicolas Capens <nicolascapens@google.com> 1613793217 +0000

Simplify If/Else lowering

Previously we would append the 'begin' basic block with the conditional
branch only once we know whether or not there's a 'false' block separate
from the 'end' block (namely when executing the Else statement). We can
instead treat the 'false' block as the 'end' block itself when no Else
is encountered and simply continue emitting instructions in this block.

Note this removes the need to explicitly materialize all variables,
because that's taken care of by createCondBr().

Bug: b/180131694
Change-Id: I5c5c4373d1dc02991feffa69d4089b2c12054dc0
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/52988
Kokoro-Result: kokoro <noreply+kokoro@google.com>
Tested-by: Nicolas Capens <nicolascapens@google.com>
Presubmit-Ready: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>