[PATCH] D110621: [llvm-locstats] Report an error if overflow was detected by llvm-dwarfdump

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/utils/llvm-locstats/llvm-locstats.py:196-198
+    if json_parsed[name] == 'overflowed':
+      print ('error: "' + name + '" field overflowed.')
+      sys.exit(1)
----------------
djtodoro wrote:
> dblaikie wrote:
> > Should it stop after the first overflowed field, or continue to print other stats?
> Some stat fields are being calculated from the other, so I guess it is better not to show any stats, than to show wrong stats.
I'd figure not to present incorrect stats - but render those tainted stats as tainted/invalid/incomputable in some way. (worst case they could say "overflow" too, though that could be slightly misleading/not-the-whole-story)


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

https://reviews.llvm.org/D110621