Simplify this class by removing the result cache.
authorGabor Greif <ggreif@gmail.com>
Fri, 11 Dec 2009 15:30:07 +0000 (15:30 +0000)
committerGabor Greif <ggreif@gmail.com>
Fri, 11 Dec 2009 15:30:07 +0000 (15:30 +0000)
commitd01518630ad165e0bc7d7194abb183abfa93e0d5
tree72178151b98f0a84aadf3d0efeb928150d613d84
parenta288b2f43a6b1a87fcd9b10c5fd46f3c65c478a0
Simplify this class by removing the result cache.

This change removes the DefaultConstructible
and CopyAssignable constraints on the template
parameter T (the first one).

The second template parameter (R) is defaulted to be
identical to the first and controls the result type.
By specifying it to be (const T&) additionally the
CopyConstructible constraint on T can be removed.

This allows to use StringSwitch e.g. for llvm::Constant
instances.

Regarding the other review feedback regarding performance
because of taking pointers, this class should be completely
optimizable like before, since all methods are inline and
the pointer dereferencing and result value caching should be
possible behind the scenes by the "as-if" rule.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@91123 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/ADT/StringSwitch.h