[PATCH] Refactor VariantMatcher to use a tagged union.

Samuel Benzaquen sbenza at google.com
Tue Aug 20 11:10:41 PDT 2013

  We can't have a Matcher<T> as an out argument, because it doesn't have a default constructor.
  I could change the interface to return an llvm::Optional<Matcher<T>> instead, which doesn't require one.
  It would remove the has/get duality. The caller would always call get() and then check hasValue() in it.


More information about the cfe-commits mailing list