<div dir="ltr">Thanks!</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 11, 2016 at 10:43 PM, Simon Atanasyan via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: atanasyan<br>
Date: Fri Aug 12 00:43:42 2016<br>
New Revision: 278480<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=278480&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=278480&view=rev</a><br>
Log:<br>
[ELF][MIPS] Fix the comment<br>
<br>
Modified:<br>
    lld/trunk/ELF/Mips.cpp<br>
<br>
Modified: lld/trunk/ELF/Mips.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Mips.cpp?rev=278480&r1=278479&r2=278480&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project/lld/trunk/ELF/Mips.<wbr>cpp?rev=278480&r1=278479&r2=<wbr>278480&view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- lld/trunk/ELF/Mips.cpp (original)<br>
+++ lld/trunk/ELF/Mips.cpp Fri Aug 12 00:43:42 2016<br>
@@ -252,12 +252,15 @@ static StringRef getArchName(uint32_t Fl<br>
   }<br>
 }<br>
<br>
-// There are (arguably too) many MIPS ISAs out there. Some are compatible<br>
-// with each other and some are not. This function checks if all input<br>
-// files are compatible with each other, and if so, returns the "lowest"<br>
-// ISA flag. For example, if one object is in EF_MIPS_ARCH_3 and the<br>
-// other is in EF_MIPS_ARCH_2, it'll return EF_MIPS_ARCH_2 because it's<br>
-// older than EF_MIPS_ARCH_3.<br>
+// There are (arguably too) many MIPS ISAs out there. Their relationships<br>
+// can be represented as a forest. If all input files have ISAs which<br>
+// reachable by repeated proceeding from the single child to the parent,<br>
+// these input files are compatible. In that case we need to return "highest"<br>
+// ISA. If there are incompatible input files, we show an error.<br>
+// For example, mips1 is a "parent" of mips2 and such files are compatible.<br>
+// Output file gets EF_MIPS_ARCH_2 flag. From the other side mips3 and mips32<br>
+// are incompatible because nor mips3 is a parent for misp32, nor mips32<br>
+// is a parent for mips3.<br>
 static uint32_t getArchFlags(ArrayRef<<wbr>FileFlags> Files) {<br>
   uint32_t Ret = Files[0].Flags & (EF_MIPS_ARCH | EF_MIPS_MACH);<br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>