ed accepted this revision.
ed added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D41500#965157, @zhmu wrote:

> @ed I tried to keep things as much in line as the other 
> <os>::linker::Construct() functions do. Do you wish to stray from that path?
>
> I actually prefer to keep it like this as it's quite readable already, but 
> YMMV of course :-)


Agreed. I'll commit this patch later today as is!


Repository:
  rC Clang

https://reviews.llvm.org/D41500



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to