[PATCH] D134817: Remove the dependency between lib/DebugInfoDWARF and MC

Shubham Sandeep Rastogi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 3 20:48:42 PDT 2022


rastogishubham added inline comments.


================
Comment at: llvm/tools/llvm-gsymutil/llvm-gsymutil.cpp:288
                              "unable to create DWARF context");
-  logAllUnhandledErrors(DICtx->loadRegisterInfo(Obj), OS, "DwarfTransformer: ");
 
----------------
clayborg wrote:
> Was this not needed?
It seems like this is not tested, and the DICtx that owns the registerinfo doesn't do anything with it, so I just removed it


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

https://reviews.llvm.org/D134817



More information about the llvm-commits mailing list