As to me the patch looks good. The code becomes much shorter. But I do not think that `std::forward` is a common idiom for constructor inheritance. Might it confuse somebody? REPOSITORY rL LLVM http://reviews.llvm.org/D8616 EMAIL PREFERENCES http://reviews.llvm.org/settings/panel/emailpreferences/