[PATCH] D37852: [dwarfdump] Make .eh_frame a first class citizen

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 14 13:30:44 PDT 2017


aprantl added inline comments.


================
Comment at: include/llvm/DebugInfo/DIContext.h:160
 
-  virtual bool verify(raw_ostream &OS, unsigned DumpType = DIDT_All,
-                      DIDumpOptions DumpOpts = {}) {
+  virtual bool verify(raw_ostream &OS, DIDumpOptions DumpOpts = {}) {
     // No verifier? Just say things went well.
----------------
aprantl wrote:
> Do we want to retain the ability to verify only part of a file? E.g., `--verify --debug-info`?
To answer my rhetoric question: I think we do.


Repository:
  rL LLVM

https://reviews.llvm.org/D37852





More information about the llvm-commits mailing list