[clang] [llvm] [Offloading] Extend OffloadBinary format to support multiple metadata entries (PR #169425)

Joseph Huber via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 3 18:47:01 PST 2025


================
@@ -181,7 +195,7 @@ class OffloadFile : public OwningBinary<OffloadBinary> {
         getBinary()->getMemoryBufferRef().getBufferIdentifier());
 
     // This parsing should never fail because it has already been parsed.
-    auto NewBinaryOrErr = OffloadBinary::create(*Buffer);
+    auto NewBinaryOrErr = OffloadBinary::createV1(*Buffer);
----------------
jhuber6 wrote:

I don't think we need a versioned create method, everything should use the new version after this. Realistically we probably just want an overload that takes a single one and another that takes an array. The old function just calls the new one with an array of size one.

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


More information about the llvm-commits mailing list