[PATCH] D60990: [Driver] Support priority for multilibs

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 23 00:16:07 PDT 2019


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





More information about the cfe-commits mailing list