[llvm] r285517 - [LoopVectorize] Make interleaved-accesses analysis less conservative about

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 30 08:22:58 PDT 2016


Also, on another bot, it caused either a long delay or an infinite
loop, resulting in timeout while building TSVC:

http://lab.llvm.org:8014/builders/clang-native-arm-lnt-perf/builds/458

Something to be on the lookout, too.

--renato

On 30 October 2016 at 15:20, Renato Golin <renato.golin at linaro.org> wrote:
> On 30 October 2016 at 15:13, Nuzman, Dorit <dorit.nuzman at intel.com> wrote:
>> My guess is that the problem is with the releaseGroup() which doesn't really remove the group; so the next loop that scans over the LoadGroups encounters a Group that had already been released (and tries to release it again).
>>
>> I have a tentative fix (basically instead of having two separate scans over the LoadGroups, do all the LoadGroup checks inside a single scan over the Groups; see patch below), but I did not have time to test it yet (also I wasn't able to reproduce the failure on my platform yet...).
>
> If you send me the whole patch+fix, I can build it locally to see if
> it fixes the problem.
>
> cheers,
> --renato


More information about the llvm-commits mailing list