From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 86089] [r600g] shader failure - r600_sb::bc_finalizer::cf_peephole() when starting Second Life Date: Mon, 10 Nov 2014 21:51:33 +0000 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0612759097==" Return-path: Received: from culpepper.freedesktop.org (unknown [131.252.210.165]) by gabe.freedesktop.org (Postfix) with ESMTP id 982DF6E1EF for ; Mon, 10 Nov 2014 13:51:33 -0800 (PST) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0612759097== Content-Type: multipart/alternative; boundary="1415656293.BeCFd82E0.4068"; charset="UTF-8" --1415656293.BeCFd82E0.4068 Date: Mon, 10 Nov 2014 21:51:33 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable https://bugs.freedesktop.org/show_bug.cgi?id=3D86089 --- Comment #2 from Shawn Starr --- Bisecting came up with this as bad commit: 73dd50acf6d244979c2a657906aa56d3ac60d550 is the first bad commit commit 73dd50acf6d244979c2a657906aa56d3ac60d550 Author: Tapani P=C3=A4lli Date: Wed Aug 6 09:46:54 2014 +0300 glsl: implement switch flow control using a loop Patch removes old variable based logic for handling a break inside switch. Switch is put inside a loop so that existing infrastructure for loop flow control can be used for the switch, now also dead code elimination works properly. Possible 'continue' call inside a switch needs now special handling which is taken care of by detecting continue, breaking out and calling continue for the outside loop. v2: remove one unnecessary ir_expression (Curro) Fixes following Piglit tests: fs-exec-after-break.shader_test fs-conditional-break.shader_test No Piglit or es3conform regressions. Signed-off-by: Tapani P=C3=A4lli Reviewed-by: Francisco Jerez :040000 040000 eac8a47066723ff804317ef11bb006dd0bb80d0f b73ed6cc78a66c54d684635f69a358fd54f65929 M src --=20 You are receiving this mail because: You are the assignee for the bug. --1415656293.BeCFd82E0.4068 Date: Mon, 10 Nov 2014 21:51:33 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

Comment= # 2 on bug 86089<= /a> from Shawn Starr 
Bisecting came up with this as bad commit:

73dd50acf6d244979c2a657906aa56d3ac60d550 is the first bad commit
commit 73dd50acf6d244979c2a657906aa56d3ac60d550
Author: Tapani P=C3=A4lli <tapani.palli@intel.com>
Date:   Wed Aug 6 09:46:54 2014 +0300

    glsl: implement switch flow control using a loop

    Patch removes old variable based logic for handling a break inside
    switch. Switch is put inside a loop so that existing infrastructure
    for loop flow control can be used for the switch, now also dead code
    elimination works properly.

    Possible 'continue' call inside a switch needs now special handling
    which is taken care of by detecting continue, breaking out and calling
    continue for the outside loop.

    v2: remove one unnecessary ir_expression (Curro)

    Fixes following Piglit tests:

       fs-exec-after-break.shader_test
       fs-conditional-break.shader_test

    No Piglit or es3conform regressions.

    Signed-off-by: Tapani P=C3=A4lli <tapani.palli@intel.com>
    Reviewed-by: Francisco Jerez <currojerez@riseup.net>

:040000 040000 eac8a47066723ff804317ef11bb006dd0bb80d0f
b73ed6cc78a66c54d684635f69a358fd54f65929 M      src

You are receiving this mail because: =20=20=20=20=20=20
  • You are the assignee for the bug.
--1415656293.BeCFd82E0.4068-- --===============0612759097== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6Ly9saXN0 cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK --===============0612759097==--