[PATCH] D78247: Remove vptr dispatch from FoldingSet.

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 16:01:51 PDT 2020


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

lgtm



================
Comment at: llvm/include/llvm/ADT/FoldingSet.h:493
+template <class T>
+class FoldingSet final : public FoldingSetImpl<FoldingSet<T>, T> {
+  using Super = FoldingSetImpl<FoldingSet, T>;
----------------
`final` seems a bit superfluous now that it is not dynamic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78247





More information about the llvm-commits mailing list