[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