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

Bataev, Alexey via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 17 21:02:42 PDT 2015


Hans, everything is fine.

Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team

17.08.2015 21:30, Hans Wennborg пишет:
> 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