[PATCH] D132747: [llvm][ADT] Overload output stream operator `<<` for `StringMapEntry` and `StringMap`.

[weiyi via Phabricator via llvm-commits]

wyt added inline comments.


================
Comment at: llvm/include/llvm/Testing/ADT/StringMap.h:26
+  if (M.empty()) {
+    return OS << "{ }";
+  }
----------------
ymandel wrote:
> Is there a standard representation for empty maps and, if so, is this it? 
I'm not aware of one, imo { } seemed pretty representative of maps.
Do you have a preference?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132747