[llvm] Add --offoading option to llvm-readobj (PR #143342)

[James Henderson via llvm-commits]

================
@@ -230,4 +232,21 @@ void ObjDumper::printSectionsAsHex(const object::ObjectFile &Obj,
   }
 }
 
+// TODO: add proper error handling.
+void ObjDumper::printOffloading(const object::ObjectFile &Obj) {
+  // we can use an argument to let user select which offloading section they
+  // want to print. but for now, we're hardcoding ELF and "hip_fatbin".
+  assert((Obj.isELF() || Obj.isCOFF()) && "Invalid file type");
+
+  SmallVector<llvm::object::OffloadBundleFatBin> Bundles;
+  if (Error Err = llvm::object::extractOffloadBundleFatBinary(Obj, Bundles))
----------------
jh7370 wrote:

Two things:
* `Err` is unused and will cause a failure in certain build configs (I forget the specific parameter).
* Why the `llvm::` prefix?

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