[Mesa-dev] [PATCH 3/5] i965/cfg: Eliminate an empty then-branch of an if/else/endif

[Ian Romanick]

On 02/26/2016 09:32 AM, Matt Turner wrote:
> On Thu, Feb 25, 2016 at 2:40 PM, Ian Romanick <idr at freedesktop.org> wrote:
>> From: Ian Romanick <ian.d.romanick at intel.com>
>>
>> On BDW,
>>
>> total instructions in shared programs: 8448571 -> 8448367 (-0.00%)
>> instructions in affected programs: 21000 -> 20796 (-0.97%)
>> helped: 116
>> HURT: 0
>>
>> Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
>> ---
>>  src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp | 18 ++++++++++++++++++
>>  1 file changed, 18 insertions(+)
>>
>> diff --git a/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp b/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp
>> index 7aa72b1..149596f 100644
>> --- a/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp
>> +++ b/src/mesa/drivers/dri/i965/brw_dead_control_flow.cpp
>> @@ -34,6 +34,7 @@
>>   *   - if/endif
>>   *   . else in else/endif
>>   *   - if/else/endif
>> + *   - then in if/else/endif
> 
> This list has been "<instruction> in <instruction sequence>" so this
> line should be "else in if/else"

Ok.  Also... is there a reason 'else in else/endif' starts with .
instead of a -?