[LLVMdev] Parallel Loop Metadata

[Redmond, Paul]

Can we summarize this thread and the final recommendation? :P 

paul

On 2013-02-12, at 1:12 PM, Nadav Rotem wrote:

> 
> On Feb 12, 2013, at 9:02 AM, Tobias Grosser <tobias at grosser.es> wrote:
> 
>> On 02/12/2013 05:54 PM, Nadav Rotem wrote:
>>> 
>>>> I have the feeling option 2) does not work for you, but I don't yet understand your reasons.
>>> 
>>> My inclination to prefer #1 is due to its simplicity. But, if #1 does not work because it creates a correctness problems then #2 is the only option that is left on the table.
>> 
>> Unfortunately. :-(
>> 
>> Does your reply mean you agree that option 2) should be taken?
> 
> Yes. I think that there is a consensus that this is the right approach.   
> 
>> In case you do, how should we proceed? Should Pekka submit his patch for a final pre-commit review?
> 
> I think so. Pekka's patch from Jan 31st looks good to me.  The is bug in the metadata handling in the vectorizer that we don't check that the load/store annotation matches the loop metadata. 
> 
>> 
>> If option 2) is OK, Paul could also try to produce this metadate from his #pragma ivdep parser.
>> 
> 
> That would be wonderful. 
> 
> Thanks,
> Nadav
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev