[PATCH] D82581: [ADT] Support const-qualified unique_functions

Sam McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 25 10:46:47 PDT 2020


sammccall created this revision.
sammccall added a reviewer: kadircet.
Herald added subscribers: llvm-commits, dexonsmith.
Herald added a project: LLVM.

This technique should extend to rvalue-qualified etc, but I didn't add any.
I removed "volatile" from the future plans, which seems... speculative at best.

While here I moved the callbacks object out of the constructor into a
variable template, which I believe addresses the fixme there about unused
objects.

(I'm not a template guru, so it's always possible the old version was designed
for compile-time performance in a way I'm missing)


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D82581

Files:
  llvm/include/llvm/ADT/FunctionExtras.h
  llvm/unittests/ADT/FunctionExtrasTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82581.273445.patch
Type: text/x-patch
Size: 14448 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200625/c7624ba8/attachment.bin>


More information about the llvm-commits mailing list