[PATCH] D14156: [X86] Enable shrink-wrapping by default

[Quentin Colombet via llvm-commits]

qcolombet updated this revision to Diff 39262.
qcolombet added a comment.

Update patch after PR24193 is fixed, i.e., now the Win64 test case did not show any changes.


Repository:
  rL LLVM

http://reviews.llvm.org/D14156

Files:
  lib/Target/X86/X86FrameLowering.h
  test/CodeGen/X86/tail-opts.ll

Index: lib/Target/X86/X86FrameLowering.h
===================================================================
--- lib/Target/X86/X86FrameLowering.h
+++ lib/Target/X86/X86FrameLowering.h
@@ -125,6 +125,11 @@
   /// \p MBB will be correctly handled by the target.
   bool canUseAsEpilogue(const MachineBasicBlock &MBB) const override;
 
+  /// Returns true if the target will correctly handle shrink wrapping.
+  bool enableShrinkWrapping(const MachineFunction &MF) const override {
+    return true;
+  }
+
   /// Wraps up getting a CFI index and building a MachineInstr for it.
   void BuildCFI(MachineBasicBlock &MBB, MachineBasicBlock::iterator MBBI,
                 DebugLoc DL, MCCFIInstruction CFIInst) const;
Index: test/CodeGen/X86/tail-opts.ll
===================================================================
--- test/CodeGen/X86/tail-opts.ll
+++ test/CodeGen/X86/tail-opts.ll
@@ -277,8 +277,8 @@
 
 ; CHECK-LABEL: foo:
 ; CHECK:        callq func
-; CHECK-NEXT: .LBB4_2:
 ; CHECK-NEXT:   popq
+; CHECK-NEXT: .LBB4_2:
 ; CHECK-NEXT:   ret
 
 define void @foo(i1* %V) nounwind {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14156.39262.patch
Type: text/x-patch
Size: 1089 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151104/a2b0c070/attachment.bin>