<div dir="ltr">Thank you for doing this, and sorry for the wrong assumption.<div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 24, 2016 at 9:03 AM, Rafael Espindola 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: rafael<br>
Date: Tue May 24 11:03:27 2016<br>
New Revision: 270573<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=270573&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=270573&view=rev</a><br>
Log:<br>
Fix a wrong assumption.<br>
<br>
Added:<br>
    lld/trunk/test/ELF/eh-frame-multilpe-cie.s<br>
Modified:<br>
    lld/trunk/ELF/OutputSections.cpp<br>
<br>
Modified: lld/trunk/ELF/OutputSections.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/OutputSections.cpp?rev=270573&r1=270572&r2=270573&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/OutputSections.cpp?rev=270573&r1=270572&r2=270573&view=diff</a><br>
==============================================================================<br>
--- lld/trunk/ELF/OutputSections.cpp (original)<br>
+++ lld/trunk/ELF/OutputSections.cpp Tue May 24 11:03:27 2016<br>
@@ -964,14 +964,6 @@ CieRecord *EhOutputSection<ELFT>::addCie<br>
   return Cie;<br>
 }<br>
<br>
-template <class ELFT> static void validateFde(SectionPiece &Piece) {<br>
-  // We assume that all FDEs refer the first CIE in the same object file.<br>
-  const endianness E = ELFT::TargetEndianness;<br>
-  uint32_t ID = read32<E>(Piece.Data.data() + 4);<br>
-  if (Piece.InputOff + 4 - ID != 0)<br>
-    fatal("invalid CIE reference");<br>
-}<br>
-<br>
 // There is one FDE per function. Returns true if a given FDE<br>
 // points to a live function.<br>
 template <class ELFT><br>
@@ -998,19 +990,30 @@ template <class ELFT><br>
 template <class RelTy><br>
 void EhOutputSection<ELFT>::addSectionAux(EhInputSection<ELFT> *Sec,<br>
                                           ArrayRef<RelTy> Rels) {<br>
-  SectionPiece &CiePiece = Sec->Pieces[0];<br>
-  // The empty record is the end marker.<br>
-  if (CiePiece.Data.size() == 4)<br>
-    return;<br>
-<br>
-  CieRecord *Cie = addCie(CiePiece, Sec, Rels);<br>
-<br>
-  for (size_t I = 1, End = Sec->Pieces.size(); I != End; ++I) {<br>
-    SectionPiece &FdePiece = Sec->Pieces[I];<br>
-    validateFde<ELFT>(FdePiece);<br>
-    if (!isFdeLive(FdePiece, Sec, Rels))<br>
+  const endianness E = ELFT::TargetEndianness;<br>
+<br>
+  DenseMap<size_t, CieRecord *> OffsetToCie;<br>
+  for (size_t I = 0, End = Sec->Pieces.size(); I != End; ++I) {<br>
+    SectionPiece &Piece = Sec->Pieces[I];<br></blockquote><div><br></div><div>You can use range-based for here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+    // The empty record is the end marker.<br>
+    if (Piece.Data.size() == 4)<br>
+      continue;<br>
+<br>
+    size_t Offset = Piece.InputOff;<br>
+    uint32_t ID = read32<E>(Piece.Data.data() + 4);<br>
+    if (ID == 0) {<br>
+      OffsetToCie[Offset] = addCie(Piece, Sec, Rels);<br>
+      continue;<br>
+    }<br>
+<br>
+    uint32_t CieOffset = Offset + 4 - ID;<br>
+    CieRecord *Cie = OffsetToCie[CieOffset];<br>
+    if (!Cie)<br>
+      fatal("invalid CIE reference");<br>
+<br>
+    if (!isFdeLive(Piece, Sec, Rels))<br>
       continue;<br>
-    Cie->FdePieces.push_back(&FdePiece);<br>
+    Cie->FdePieces.push_back(&Piece);<br>
     NumFdes++;<br>
   }<br>
 }<br>
<br>
Added: lld/trunk/test/ELF/eh-frame-multilpe-cie.s<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/test/ELF/eh-frame-multilpe-cie.s?rev=270573&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/test/ELF/eh-frame-multilpe-cie.s?rev=270573&view=auto</a><br>
==============================================================================<br>
--- lld/trunk/test/ELF/eh-frame-multilpe-cie.s (added)<br>
+++ lld/trunk/test/ELF/eh-frame-multilpe-cie.s Tue May 24 11:03:27 2016<br>
@@ -0,0 +1,12 @@<br>
+// RUN: llvm-mc %s -o %t.o -filetype=obj -triple=x86_64-pc-linux<br>
+// RUN: ld.lld --eh-frame-hdr %t.o -o %t.so -shared<br>
+// We would fail to parse multiple cies in the same file.<br>
+<br>
+        .cfi_startproc<br>
+        .cfi_personality 0x9b, foo<br>
+        .cfi_endproc<br>
+<br>
+        .cfi_startproc<br>
+        .cfi_endproc<br>
+<br>
+foo:<br>
<br>
<br>
_______________________________________________<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/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>