[PATCH] D28014: Support: Add YAML I/O support for string maps.

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added inline comments.


================
Comment at: llvm/trunk/lib/Support/YAMLTraits.cpp:389-393
+  for (std::string &K : ValidKeys) {
+    if (Key == K)
       return true;
   }
   return false;
----------------
majnemer wrote:
> Isn't this: `return is_contained(ValidKeys, Key);`?
Sweet, I didn't know about `llvm::is_contained`!


Repository:
  rL LLVM

https://reviews.llvm.org/D28014