| Summary: | opencv_test_imgproc crash when test OCL_Imgproc/HoughLines.RealImage | ||
|---|---|---|---|
| Product: | Beignet | Reporter: | Yan Wang <yan.wang> |
| Component: | Beignet | Assignee: | Zhigang Gong <zhigang.gong> |
| Status: | CLOSED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | medium | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux (All) | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
| Attachments: | error log and back trace for debugging. | ||
opencv: master(3.0-dev) commit e40567eaeeac42dd9dfd4791812fa866df84c206 beginet: master commit 1ea7977ee84df4c7607c82f0846307c63f8364df Please verify whether the patch, I just sent to the mail list, could fix this bug. Thanks. commit 2234c334af6c0254969e0748c7ba14ef2c8a009e Author: Zhigang Gong <zhigang.gong@intel.com> Date: Fri Oct 24 09:55:34 2014 +0800 GBE: handle dead loop BBs in liveness analysis. Considering the following CFG, our previous liveness analysis will only back traverse from the exit point BB, thus the BB 6 and 7 will not be handled. 4--- | | | | 5 --|---> 10 ---> ret | | | | 6<-- | |<-- 7 | | | --- Although the CFG looks not a normal application as once it goes to block 6, it will enter a dead loop and will never return, we still need to compile it successfully. This patch is to fix the bug at: https://bugs.freedesktop.org/show_bug.cgi?id=85362 Signed-off-by: Zhigang Gong <zhigang.gong@intel.com> I tried your patch and the bug is fixed. Thanks. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.
Created attachment 108293 [details] error log and back trace for debugging. prerequisite： unset OPENCV_TEST_DATA_PATH Run: opecv-test-imgproc --gtest_filter=OCL_Imgproc/HoughLines.RealImage* Error: [ RUN ] OCL_Imgproc/HoughLines.RealImage/0 ASSERTION FAILED: defIt != bbIt->second->end() && defIt->second != NULL at file /home/yanwang/beignet/backend/src/ir/value.cpp, function gbe::ir::LiveOutSet::RegDefSet& gbe::ir::LiveOutSet::getDefSet(const gbe::ir::BasicBlock*, gbe::ir::Register), line 91