[llvm] r303490 - COFF: migrate def parser from LLD to LLVM [1/2]
Martell Malone via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 8 17:02:29 PDT 2017
LGTM
Removing the cast with a note about naming makes a lot of sense here.
Thanks Galina for finding a reasonable solution to fix the warnings.
Rui, I can follow up with a commit on the naming if you have a suggestion?
Best,
Martell
On Fri, Jun 9, 2017 at 12:38 AM, Galina Kistanova <gkistanova at gmail.com>
wrote:
> I have committed the fix as r305029.
>
> Naming there looks a bit odd, but this is for somebody else to address. :)
>
> Thanks
>
> Galina
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170609/6309cd11/attachment.html>
More information about the llvm-commits
mailing list