Unrelated to this specific patch, however: On Thu, May 15, 2014 at 4:17 AM, "C. Bergström" <[email protected]>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.
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
