[PATCH] [OPENMP] Loop canonical form analysis (Sema)

Chandler Carruth chandlerc at google.com
Thu May 22 21:55:49 PDT 2014


Unrelated to this specific patch, however:

On Thu, May 15, 2014 at 4:17 AM, "C. Bergström" <cbergstrom at pathscale.com>wrote:
>
> LGTM - maybe give it 1-2 more days for others to have a chance to review
> and if no objections push - Thanks
>

Please don't mark patches as looking good and encouraging the contributors
to commit for areas of the code and/or project you have no familiarity
with. I understand that you're quite familiar with OpenMP, but as you
aren't a regular Clang contributor (very few patches submitted, none I
could find committed directly) and aren't one of the maintainers or code
owners of Sema, it seems better to leave the final review to others.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140522/bf96af97/attachment.html>


More information about the cfe-commits mailing list