[PATCH] LLD: Inherit constructors. No functionality change.
Rui Ueyama
ruiu at google.com
Wed Mar 25 13:31:51 PDT 2015
Yup, I'd like to hear from more audiences about their opinions.
On Wed, Mar 25, 2015 at 1:28 PM, Simon Atanasyan <simon at atanasyan.com>
wrote:
> 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/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150325/1c3255fc/attachment.html>
More information about the llvm-commits
mailing list