dblaikie added a comment.

Could you upload this with full context ( 
https://llvm.org/docs/Phabricator.html#id4 mentions how to do this)?

Does this still have an OpenMP special case? The production code changes don't 
seem to mention OpenMP anymore, but there's still a lot of test updates for 
OpenMP - what are they for?

@rsmith could you take a look at this - the pending instantiations stuff isn't 
something I'm sufficiently familiar with to feel comfortable signing off on 
this alone.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69585/new/

https://reviews.llvm.org/D69585



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D69585: P... David Blaikie via Phabricator via cfe-commits

Reply via email to