[llvm] [Offload] Allow querying the size of globals (PR #147698)

[Ross Brunton via llvm-commits]

================
@@ -37,20 +37,33 @@ using namespace llvm::object;
 /// Common abstraction for globals that live on the host and device.
 /// It simply encapsulates the symbol name, symbol size, and symbol address
 /// (which might be host or device depending on the context).
+/// Both size and address may be absent, and can be populated with
+// getGlobalMetadataFromDevice/Image.
 class GlobalTy {
   // NOTE: Maybe we can have a pointer to the offload entry name instead of
   // holding a private copy of the name as a std::string.
   std::string Name;
-  uint32_t Size;
-  void *Ptr;
+  std::optional<uint32_t> Size;
+  std::optional<void *> Ptr;
----------------
RossBrunton wrote:

It's possible to set the size but not the pointer; the `getGlobalMetadata` functions use this to verify that the size is correct.

https://github.com/llvm/llvm-project/pull/147698