[LLVMdev] [PATCH] parallel loop awareness to the LoopVectorizer
Redmond, Paul
paul.redmond at intel.com
Mon Jan 28 11:23:09 PST 2013
It sounds like a good idea to move the method in to Loop.
Is there a naming scheme for metadata? I think llvm.loop.* would be helpful for loop-specific metadata. As for parallel I think it is a little too generic. If ivdep are the semantics you're going for I'd use that.
paul
On 2013-01-28, at 12:03 PM, Pekka Jääskeläinen wrote:
> On 01/28/2013 06:45 PM, Nadav Rotem wrote:
>> I am okay with this patch, assuming that you follow the review of Tobias
>> and Renato and provide a separate patch for the min-iter-count and a few
>> test cases.
>
> OK. Any opinions on the location of the isParallelLoop() check? Shall I
> put it to Loop so it is more widely accessible? I.e. Loop->isParallel().
>
> --
> Pekka
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
More information about the llvm-dev
mailing list