[PATCH] D106707: [OpenMP] Introduce RAII to protect certain RTL calls from DCE

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LG, two nits.



================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:1787
+    GlobalValue::LinkageTypes LinkageType;
+  };
+
----------------
no need to store OMPInfoCache


================
Comment at: llvm/lib/Transforms/IPO/OpenMPOpt.cpp:1818-1821
+    ExternalizationRAII(OMPInfoCache,
+                        OMPRTL___tgt_target_data_begin_mapper_wait);
+    ExternalizationRAII(OMPInfoCache,
+                        OMPRTL___tgt_target_data_begin_mapper_issue);
----------------
`_tgt_...` are not device runtime functions, no need to have them here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106707