<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">LGTM.<br>
      <br>
      On 6/20/2014 9:02 AM, Tim Northover wrote:<br>
    </div>
    <blockquote
cite="mid:CAFHTzfKEsGjy-Q5E1q4YUdPmF=xU-gM+GVkkT6Ed-=QdnzZe5A@mail.gmail.com"
      type="cite">
      <pre wrap="">Hi,

I noticed that MachOLinkerContext has an _outputFileType variable
which shadows one in the parent LinkerContext class. This confused me
for quite a while, so I've attached a patch which renames it to
"_outputMachOType" (similarly to ELF).

Is this OK to commit, or did someone give them the same name intentionally?

Cheers.

Tim.
</pre>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
llvm-commits mailing list
<a class="moz-txt-link-abbreviated" href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>
<a class="moz-txt-link-freetext" href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a>
</pre>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation</pre>
  </body>
</html>