[PATCH] D27001: [LLVM] [ADT] Fix some Clang-tidy modernize-use-default and Include What You Use warnings; other minor fixes

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 22 15:29:24 PST 2016


mehdi_amini accepted this revision.
mehdi_amini added a comment.
This revision is now accepted and ready to land.

I didn't quite get why you put it up for review? Why does the "ADT takes a long time to build" matter?



================
Comment at: include/llvm/ADT/StringMap.h:20
 #include "llvm/Support/PointerLikeTypeTraits.h"
+#include <cassert>
+#include <cstdint>
----------------
There are no assert in this file, why is this needed?


Repository:
  rL LLVM

https://reviews.llvm.org/D27001





More information about the llvm-commits mailing list