[PATCH] Fix tuple<A> to A conversion in SFINAE for tuple(_Up...) constructor

Eric Fiselier eric at efcs.ca
Fri May 1 00:04:29 PDT 2015


Tuple is tricky. If there is a bug in tuple you can bet that SFINAE is both the problem and the solution. constexpr constructors are becoming a frequent problem in the tuple SFINAE because they tend to cause hard compile errors. The SFINAE you added could easily introduce an almost identical bug in a different scenario.

Another reason I'm so hesitant to move forward with this patch is because they essentially contradicts http://reviews.llvm.org/D8178.

The first step to solving both of these bugs is to drastically increase the test coverage of tuple. The tests need to cover a much wider range of types than they currently do. Until then I think both these patches will need to wait in limbo.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9085

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list