<div dir="ltr">When you revert a commit, please write a reason as to why you are reverting as a commit message.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 15, 2018 at 7:38 AM Aleksandar Beserminji via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: abeserminji<br>
Date: Mon Oct 15 07:36:48 2018<br>
New Revision: 344515<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=344515&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=344515&view=rev</a><br>
Log:<br>
[mips][micromips] Revert "Fix overlaping FDEs error"<br>
<br>
This reverts r344511.<br>
<br>
Removed:<br>
    llvm/trunk/test/DebugInfo/Mips/eh_frame.ll<br>
Modified:<br>
    llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.cpp<br>
    llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.h<br>
<br>
Modified: llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.cpp?rev=344515&r1=344514&r2=344515&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.cpp?rev=344515&r1=344514&r2=344515&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.cpp (original)<br>
+++ llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.cpp Mon Oct 15 07:36:48 2018<br>
@@ -15,7 +15,6 @@<br>
 #include "llvm/MC/MCAssembler.h"<br>
 #include "llvm/MC/MCCodeEmitter.h"<br>
 #include "llvm/MC/MCContext.h"<br>
-#include "llvm/MC/MCDwarf.h"<br>
 #include "llvm/MC/MCInst.h"<br>
 #include "llvm/MC/MCObjectWriter.h"<br>
 #include "llvm/MC/MCSymbolELF.h"<br>
@@ -54,22 +53,6 @@ void MipsELFStreamer::EmitInstruction(co<br>
   createPendingLabelRelocs();<br>
 }<br>
<br>
-void MipsELFStreamer::EmitCFIStartProcImpl(MCDwarfFrameInfo &Frame) {<br>
-  Frame.Begin = getContext().createTempSymbol();<br>
-  MCELFStreamer::EmitLabel(Frame.Begin);<br>
-}<br>
-<br>
-MCSymbol *MipsELFStreamer::EmitCFILabel() {<br>
-  MCSymbol *Label = getContext().createTempSymbol("cfi", true);<br>
-  MCELFStreamer::EmitLabel(Label);<br>
-  return Label;<br>
-}<br>
-<br>
-void MipsELFStreamer::EmitCFIEndProcImpl(MCDwarfFrameInfo &Frame) {<br>
-  Frame.End = getContext().createTempSymbol();<br>
-  MCELFStreamer::EmitLabel(Frame.End);<br>
-}<br>
-<br>
 void MipsELFStreamer::createPendingLabelRelocs() {<br>
   MipsTargetELFStreamer *ELFTargetStreamer =<br>
       static_cast<MipsTargetELFStreamer *>(getTargetStreamer());<br>
<br>
Modified: llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.h<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.h?rev=344515&r1=344514&r2=344515&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.h?rev=344515&r1=344514&r2=344515&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.h (original)<br>
+++ llvm/trunk/lib/Target/Mips/MCTargetDesc/MipsELFStreamer.h Mon Oct 15 07:36:48 2018<br>
@@ -25,7 +25,6 @@ namespace llvm {<br>
 class MCAsmBackend;<br>
 class MCCodeEmitter;<br>
 class MCContext;<br>
-class MCDwarfFrameInfo;<br>
 class MCSubtargetInfo;<br>
<br>
 class MipsELFStreamer : public MCELFStreamer {<br>
@@ -61,12 +60,6 @@ public:<br>
   void EmitValueImpl(const MCExpr *Value, unsigned Size, SMLoc Loc) override;<br>
   void EmitIntValue(uint64_t Value, unsigned Size) override;<br>
<br>
-  // Overriding these functions allows us to avoid recording of these labels<br>
-  // in EmitLabel and later marking them as microMIPS.<br>
-  void EmitCFIStartProcImpl(MCDwarfFrameInfo &Frame) override;<br>
-  void EmitCFIEndProcImpl(MCDwarfFrameInfo &Frame) override;<br>
-  MCSymbol *EmitCFILabel() override;<br>
-<br>
   /// Emits all the option records stored up until the point it's called.<br>
   void EmitMipsOptionRecords();<br>
<br>
<br>
Removed: llvm/trunk/test/DebugInfo/Mips/eh_frame.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Mips/eh_frame.ll?rev=344514&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/Mips/eh_frame.ll?rev=344514&view=auto</a><br>
==============================================================================<br>
--- llvm/trunk/test/DebugInfo/Mips/eh_frame.ll (original)<br>
+++ llvm/trunk/test/DebugInfo/Mips/eh_frame.ll (removed)<br>
@@ -1,38 +0,0 @@<br>
-; RUN: llc -mtriple mips-unknown-linux-gnu -mattr=+micromips -O3 -filetype=obj -o - %s | llvm-readelf -r | FileCheck %s<br>
-<br>
-; CHECK: .rel.eh_frame<br>
-; CHECK: DW.ref.__gxx_personality_v0<br>
-; CHECK-NEXT: .text<br>
-; CHECK-NEXT: .gcc_except_table<br>
-<br>
-@_ZTIi = external constant i8*<br>
-<br>
-define dso_local i32 @main() local_unnamed_addr personality i8* bitcast (i32 (...)* @__gxx_personality_v0 to i8*) {<br>
-entry:<br>
-  %exception.i = tail call i8* @__cxa_allocate_exception(i32 4) nounwind<br>
-  %0 = bitcast i8* %exception.i to i32*<br>
-  store i32 5, i32* %0, align 16<br>
-  invoke void @__cxa_throw(i8* %exception.i, i8* bitcast (i8** @_ZTIi to i8*), i8* null) noreturn<br>
-          to label %.noexc unwind label %return<br>
-<br>
-.noexc:<br>
-  unreachable<br>
-<br>
-return:<br>
-  %1 = landingpad { i8*, i32 }<br>
-          catch i8* null<br>
-  %2 = extractvalue { i8*, i32 } %1, 0<br>
-  %3 = tail call i8* @__cxa_begin_catch(i8* %2) nounwind<br>
-  tail call void @__cxa_end_catch()<br>
-  ret i32 0<br>
-}<br>
-<br>
-declare i32 @__gxx_personality_v0(...)<br>
-<br>
-declare i8* @__cxa_begin_catch(i8*) local_unnamed_addr<br>
-<br>
-declare void @__cxa_end_catch() local_unnamed_addr<br>
-<br>
-declare i8* @__cxa_allocate_exception(i32) local_unnamed_addr<br>
-<br>
-declare void @__cxa_throw(i8*, i8*, i8*) local_unnamed_addr<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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>