[Openmp-commits] [PATCH] D93293: [OpenMP][Libomptarget] Introduce changes to support remote plugin

Sun Jan 31 10:06:19 PST 2021

jdoerfert added a comment.

In D93293#2532699 <https://reviews.llvm.org/D93293#2532699>, @sylvestre.ledru wrote:

> @atmnpatel
> Seems that this change caused:
> https://bugs.llvm.org/show_bug.cgi?id=48973

I don't understand when/why an `RTLsTy` object is copied at all. @atmnpatel What if we move `LoadRTLs` into a helper function that is not a member, pass a pointer into it instead. That might solve the issue. Or, pass `&PM.RTLs` instead, to match the former `call_once` call. Can you take a look?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93293