[PATCH] D69585: PerformPendingInstatiations() already in the PCH

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 13 11:50:09 PDT 2020


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





More information about the cfe-commits mailing list