[Lldb-commits] [PATCH] D130580: Refactor string conversion for InstructionControlFlowKind enum

[Jakob Johnson via Phabricator via lldb-commits]

jj10306 added inline comments.


================
Comment at: lldb/source/Core/Disassembler.cpp:930
+  }
+}
+
----------------
Thoughts on adding a default case with a string that indicates that no string conversion was implemented for the provided variant?
This would potentially make it easier for someone to realize they didn't add a case here for a newly added variant of InstructionControlFlowKind. Otherwise undefined data would be returned if a new case isn't added for a new variant.
wdyt?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130580