[PATCH] D73484: [llvm-readobj] - Improve error message reported by DynRegionInfo.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 01:30:32 PST 2020


grimar marked 2 inline comments as done.
grimar added inline comments.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:139
+  /// Error prefix. Used for error reporting to provide more information.
+  std::string InfoContext;
+  /// Region size name. Used for error reporting.
----------------
jhenderson wrote:
> Having seen the change, I don't think I like `InfoContext`, sorry! How about just `Context`?
Sounds good to me. Will apply, thanks!


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

https://reviews.llvm.org/D73484





More information about the llvm-commits mailing list