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

David Majnemer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 3 21:33:58 PST 2017


majnemer 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;
----------------
Isn't this: `return is_contained(ValidKeys, Key);`?


Repository:
  rL LLVM

https://reviews.llvm.org/D28014





More information about the llvm-commits mailing list