<div dir="ltr">Ah, so you don't have a commit access? Do you want me to check in all these patches for x32?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 12, 2016 at 4:21 PM, H.J. Lu <span dir="ltr"><<a href="mailto:hjl.tools@gmail.com" target="_blank">hjl.tools@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Jul 12, 2016 at 4:14 PM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> ruiu accepted this revision.<br>
> ruiu added a comment.<br>
> This revision is now accepted and ready to land.<br>
><br>
> LGTM with a nit.<br>
><br>
><br>
> ================<br>
> Comment at: ELF/Target.cpp:239-246<br>
> @@ -238,3 +238,10 @@<br>
>    case EM_X86_64:<br>
> -    return new X86_64TargetInfo();<br>
> +    switch (Config->EKind) {<br>
> +    case ELF32LEKind:<br>
> +      return new X86_64TargetInfo<ELF32LE>();<br>
> +    case ELF64LEKind:<br>
> +      return new X86_64TargetInfo<ELF64LE>();<br>
> +    default:<br>
> +      fatal("unsupported X86-64 target");<br>
> +    }<br>
>    }<br>
> ----------------<br>
> This seems a bit too defensive. I'd simply do<br>
><br>
>     if (Config->EKind == ELF32LEKind)<br>
>       return new X86_64TargetInfo<ELF32LE>();<br>
>     return new X86_64TargetInfo<ELF64LE>();<br>
><br>
><br>
> Repository:<br>
>   rL LLVM<br>
><br>
> <a href="http://reviews.llvm.org/D22287" rel="noreferrer" target="_blank">http://reviews.llvm.org/D22287</a><br>
><br>
<br>
</div></div>Here is the updated patch.  Please check it in for me.<br>
<br>
Thanks.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
H.J.<br>
</font></span></blockquote></div><br></div>