[llvm] Fix compress/decompress in LLVM Offloading API (PR #150064)

David Salinas via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 6 12:02:04 PDT 2025


================
@@ -259,87 +291,278 @@ static std::string formatWithCommas(unsigned long long Value) {
   return Num;
 }
 
-llvm::Expected<std::unique_ptr<llvm::MemoryBuffer>>
-CompressedOffloadBundle::decompress(llvm::MemoryBufferRef &Input,
-                                    bool Verbose) {
-  StringRef Blob = Input.getBuffer();
+Expected<std::unique_ptr<MemoryBuffer>>
+CompressedOffloadBundle::compress(compression::Params P,
----------------
david-salinas wrote:

You're right, technically we don't need "compress" ... yet.  The tools that are currently using this API only call "decompress".  Though my intention is to have other tools, like clang-offload-bundler, use this API to access HIP fatbin bundles.  So, I've added both the compress and decompress functionality.

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


More information about the llvm-commits mailing list