[PATCH] D44570: [dwarf] Unify unknown dwarf enum formatting code

[Pavel Labath via Phabricator via llvm-commits]

labath added inline comments.


================
Comment at: include/llvm/BinaryFormat/Dwarf.h:597
+    if (Str.empty()) {
+      OS << "DW_" << dwarf::EnumTraits<Enum>::Type << "_Unknown_"
+         << llvm::format("%x", E);
----------------
JDevlieghere wrote:
> Is the capitalization of Unknown on purpose (to differentiate with valid enum values)?
That's what the existing code was doing. I personally have no preference for either..


Repository:
  rL LLVM

https://reviews.llvm.org/D44570