hfinkel added a comment.

In https://reviews.llvm.org/D48808#1149862, @Meinersbur wrote:

> In https://reviews.llvm.org/D48808#1149845, @hfinkel wrote:
>
> > We specifically defined the metadata to support nested loops. The LangRef 
> > says, "The llvm.mem.parallel_loop_access metadata refers to a loop 
> > identifier, or metadata containing a list of loop identifiers for nested 
> > loops." To handle nested loops, we need to make the instruction metadata 
> > point to a list of loop IDs.
>
>
> Thanks for pointing me to it. I've not seen parallel_loop_access pointing to 
> a list, so I didn't know it was possible.


Yea, well, we defined it to be possible. As to whether the backend will 
actually correctly read the metadata in that format, I don't know.


Repository:
  rC Clang

https://reviews.llvm.org/D48808



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to