r245041 - [OPENMP] Fix for http://llvm.org/PR24371: Assert failure compiling blender 2.75.

Hans Wennborg via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 17 11:30:31 PDT 2015


Merged in r245229.

There were merge conflicts in StmtOpenMP.h because we don't have
r244209 on the branch. I think I got it right, but I'd appreciate if
someone could double check
http://llvm.org/viewvc/llvm-project?rev=245229&view=rev

Thanks,
Hans

On Sun, Aug 16, 2015 at 8:55 PM, Bataev, Alexey <a.bataev at hotmail.com> wrote:
> Yes, please, forget to tell you. :)
>
> Best regards,
> Alexey Bataev
> =============
> Software Engineer
> Intel Compiler Team
>
> 14.08.2015 19:03, Hans Wennborg пишет:
>
>> On Fri, Aug 14, 2015 at 5:25 AM, Alexey Bataev via cfe-commits
>> <cfe-commits at lists.llvm.org> wrote:
>>>
>>> Author: abataev
>>> Date: Fri Aug 14 07:25:37 2015
>>> New Revision: 245041
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=245041&view=rev
>>> Log:
>>> [OPENMP] Fix for http://llvm.org/PR24371: Assert failure compiling
>>> blender 2.75.
>>> blender uses statements expression in condition of the loop under control
>>> of the '#pragma omp parallel for'. This condition is used several times in
>>> different expressions required for codegen of the loop directive. If there
>>> are some variables defined in statement expression, it fires an assert
>>> during codegen because of redefinition of the same variables.
>>> We have to rebuild several expression to be sure that all variables are
>>> unique.
>>
>> Should we merge this to 3.7?
>
>


More information about the cfe-commits mailing list