[PATCH] D28756: Rewrite part of how loop ID is obtained.

Xin Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 16 09:19:52 PST 2017


trentxintong added a comment.

In https://reviews.llvm.org/D28756#647368, @sanjoy wrote:

> Code-wise this looks fine, but the "If a loop has multiple backedges and all of them have the same metadata we return that metadata. Otherwise we return nullptr." bit should be documented in the LangRef.  I'll also ask you to please wait for someone familiar with how LLVM uses `!llvm.loop` to give a final go head before checking this in.


Thanks for the review. I will wait for @hfinkel to give the final approval.


https://reviews.llvm.org/D28756





More information about the llvm-commits mailing list