[llvm] r288051 - [StructurizeCFG] Use range-based for loops.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 29 14:00:04 PST 2016


Fixed in r288200.  Thank you again for this.

On Tue, Nov 29, 2016 at 11:47 AM, Justin Lebar <jlebar at google.com> wrote:
> Thank you for the testcase (and for bisecting this).  I have a fix;
> I'm just reducing this testcase now.
>
> On Tue, Nov 29, 2016 at 12:12 AM, Michel Dänzer <michel at daenzer.net> wrote:
>>
>> Hi Justin,
>>
>>
>> On 29/11/16 03:50 AM, Justin Lebar via llvm-commits wrote:
>>> Author: jlebar
>>> Date: Mon Nov 28 12:50:03 2016
>>> New Revision: 288051
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=288051&view=rev
>>> Log:
>>> [StructurizeCFG] Use range-based for loops.
>>>
>>> Reviewers: arsenm
>>>
>>> Subscribers: wdng, llvm-commits
>>>
>>> Differential Revision: https://reviews.llvm.org/D27000
>>
>> This change caused an infinite loop in StructurizeCFG::rebuildSSA when
>> running the piglit test arb_shader_image_load_store-shader-mem-barrier
>> with the radeonsi driver. You can reproduce it by feeding the attached
>> LLVM IR to
>>
>> llc -march=amdgcn -mcpu=kaveri
>>
>>
>> --
>> Earthling Michel Dänzer               |               http://www.amd.com
>> Libre software enthusiast             |             Mesa and X developer


More information about the llvm-commits mailing list