lebedev.ri added a comment.

Can this have test coverage?



================
Comment at: clang/lib/Driver/Multilib.cpp:22
 #include <cassert>
+#include <map>
 #include <string>
----------------
Doesn't seem to be used?


================
Comment at: clang/lib/Driver/Multilib.cpp:271
+  // Sort multilibs by priority and select the one with the highest priority.
+  std::sort(Filtered.begin(), Filtered.end(),
+            [](const Multilib &a, const Multilib &b) -> bool {
----------------
`llvm::sort`.


Repository:
  rC Clang

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

https://reviews.llvm.org/D60990



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to