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

Fri Mar 13 13:29:18 PDT 2020

llunak added a comment.

In D69585#1922108 <https://reviews.llvm.org/D69585#1922108>, @dblaikie wrote:

> 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?

It's been handled by b841b9e96e605bed5a1f9b846a07aae88c65ce02 <https://reviews.llvm.org/rGb841b9e96e605bed5a1f9b846a07aae88c65ce02> . The tests need updates because they test compilation both with and without PCH use, and this patch reorders templates in the output, without any functional change.

Repository:
  rC Clang

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

https://reviews.llvm.org/D69585