fhahn added a comment. Looks like a nice improvement, thanks!
================ Comment at: llvm/unittests/ADT/SmallVectorTest.cpp:867 - template <typename T, unsigned N> - static unsigned NumBuiltinElts(const SmallVector<T, N>&) { return N; } }; ---------------- This seems like an unrelated change unless I am missing something? If so, this could likely be pulled out and just submitted separately as NFC Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130268/new/ https://reviews.llvm.org/D130268 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits