Re: [Beignet] [PATCH 2/2] GBE: fix a corner case which refer to a undefined value.

2013-12-17 Thread Zhigang Gong
Behalf Of Song, Ruiling Sent: Wednesday, December 18, 2013 10:06 AM To: Gong, Zhigang; beignet@lists.freedesktop.org Cc: Gong, Zhigang Subject: Re: [Beignet] [PATCH 2/2] GBE: fix a corner case which refer to a undefined value. Really strange corner case, I thought that normally the basic blocks

Re: [Beignet] [PATCH 2/2] GBE: fix a corner case which refer to a undefined value.

2013-12-17 Thread Song, Ruiling
:26 PM To: beignet@lists.freedesktop.org Cc: Gong, Zhigang Subject: [Beignet] [PATCH 2/2] GBE: fix a corner case which refer to a undefined value. Clang/llvm may generate some code similar to the following IRs: ... (there is no definition of %7) br label 2 label1: %10 = add %7, %6 ... ret

[Beignet] [PATCH 2/2] GBE: fix a corner case which refer to a undefined value.

2013-12-16 Thread Zhigang Gong
Clang/llvm may generate some code similar to the following IRs: ... (there is no definition of %7) br label 2 label1: %10 = add %7, %6 ... ret label2: %7 = ... br label1 The value %7 is assigned after label2 but is referred at label1. >From the control flow, the IRs is valid. As th