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

[Zhigang Gong]

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