[Beignet] OpenCV Morph OCL Perf case is crash based on beignet

[yan.wang at linux.intel.com]

I confirmed it is fixed.
Thanks.

> Yan,
>
> How about this one?
>
> On Thu, Oct 23, 2014 at 12:29:34AM -0700, yan.wang at linux.intel.com wrote:
>> Sorry, I should update beignet not opencv.
>> I will try.
>>
>> Yan Wang
>>
>> > You commit seems not to be merged yet.
>> > If merged, I will try.
>> > Thanks.
>> >
>> > Yan Wang
>> >
>> >> Did you try latest git master?
>> >> It should has been fixed by commit:
>> >> commit 5417a7e2afb3f2043dbb8abbaebca5115b87ab70
>> >> Author: Zhigang Gong <zhigang.gong at intel.com>
>> >> Date:   Tue Oct 21 21:03:46 2014 +0800
>> >>
>> >>     GBE: add basic PHINode support in legalize pass.
>> >>
>> >>     Lack of the incomplete PHINode support currently.
>> >>     After this patch, we could fix all the crash cases in piglit.
>> >>
>> >>     Signed-off-by: Zhigang Gong <zhigang.gong at intel.com>
>> >>     Tested-by: "Meng, Mengmeng" <mengmeng.meng at intel.com>
>> >>
>> >>> -----Original Message-----
>> >>> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On
>> Behalf
>> >>> Of
>> >>> yan.wang at linux.intel.com
>> >>> Sent: Thursday, October 23, 2014 3:07 PM
>> >>> To: beignet at lists.freedesktop.org
>> >>> Subject: [Beignet] OpenCV Morph OCL Perf case is crash based on
>> beignet
>> >>>
>> >>> Hi, All,
>> >>>   Today I tried the latest opencv code (3.0 master) and found morph
>> >>> perf
>> >> case
>> >>> will crash when test CV_8UC1 type only.
>> >>>   The case number is the following:
>> >>>   opencv_perf_imgproc
>> >>> --gtest_filter=OCL_MorphologyExFixture_MorphologyEx.MorphologyEx/4.
>> >>>   Its operation type is MORPHY_GRADIENT.
>> >>>   I do the initial debug and found it crashed in llvm_legalize.cpp
>> line
>> >>> 516 because insn->getOpcode() couldn't get a valid op code.
>> >>>
>> >>>   %.0.i = phi i128 [ %phitmp962, %58 ], [ 0,
>> >>> %isBorder.exit.thread.thread650 ], [ 0, %19 ], [ 0, %25 ] ASSERTION
>> >> FAILED:
>> >>> false && "Illegal instruction\n"
>> >>>   at file /home/yanwang/beignet/backend/src/llvm/llvm_legalize.cpp,
>> >>> function bool gbe::Legalize::legalizeFunction(llvm::Function&), line
>> >>> 516
>> >>>
>> >>>   After asked Junyan, it seems "phi" op code hasn't been supported.
>> >>>   Thanks.
>> >>>
>> >>> Yan Wang
>> >>> _______________________________________________
>> >>> Beignet mailing list
>> >>> Beignet at lists.freedesktop.org
>> >>> http://lists.freedesktop.org/mailman/listinfo/beignet
>> >>
>> >> _______________________________________________
>> >> Beignet mailing list
>> >> Beignet at lists.freedesktop.org
>> >> http://lists.freedesktop.org/mailman/listinfo/beignet
>> >>
>> >
>> > _______________________________________________
>> > Beignet mailing list
>> > Beignet at lists.freedesktop.org
>> > http://lists.freedesktop.org/mailman/listinfo/beignet
>> >
>>
>> _______________________________________________
>> Beignet mailing list
>> Beignet at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/beignet
>