[PATCH] D67328: [llvm-readobj] Warn user when dumping not supported version of stack map (PR38278).

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 10 01:44:35 PDT 2019


MaskRay added inline comments.


================
Comment at: llvm/tools/llvm-readobj/StackMapPrinter.h:29-30
+                      "stack map entry with version " + Twine(Version) +
+                      " detected at offset " + Twine(0) + ". Only version " +
+                      Twine(SupportedVersion) + " is supported"),
+                  Filename);
----------------
grimar wrote:
> jhenderson wrote:
> > If you accept my suggestion of returning multiple supported versions, you'll need to change this message again (sorry!). I'd suggest changing the second sentence to "Supported version(s): " and then simply list the array members.
> I think just "version X is not supported" should be fine. Is it important to report a list of supported versions here? I am not sure.
`"... offset" + utohexstr(Offset)`

Offsets are usually represented as a hex number. Since the offset is always 0, you can just hard code `"offset 0x0"`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67328





More information about the llvm-commits mailing list