From 1c55fab53455f11fbd7bde69d0c5911031218e35 Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Wed, 5 Aug 2009 00:49:25 +0000 Subject: [PATCH] Don't flush the raw_ostream between each MachineFunction. These flush calls were originally put in place because errs() at one time was not unbuffered, and these print routines are commonly used with errs() for debugging. However, errs() is now properly unbuffered, so the flush calls are no longer needed. This significantly reduces the number of write(2) calls for regular asm printing when there are many small functions. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@78137 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp | 2 -- lib/Target/MSP430/MSP430AsmPrinter.cpp | 2 -- lib/Target/PowerPC/AsmPrinter/PPCAsmPrinter.cpp | 2 -- lib/Target/SystemZ/AsmPrinter/SystemZAsmPrinter.cpp | 2 -- lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp | 2 -- lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp | 2 -- 6 files changed, 12 deletions(-) diff --git a/lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp b/lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp index d1bf7007187..ab49d166ff0 100644 --- a/lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp +++ b/lib/Target/ARM/AsmPrinter/ARMAsmPrinter.cpp @@ -330,8 +330,6 @@ bool ARMAsmPrinter::runOnMachineFunction(MachineFunction &MF) { // Emit post-function debug information. DW->EndFunction(&MF); - O.flush(); - return false; } diff --git a/lib/Target/MSP430/MSP430AsmPrinter.cpp b/lib/Target/MSP430/MSP430AsmPrinter.cpp index e357761705a..2da7cdd61ca 100644 --- a/lib/Target/MSP430/MSP430AsmPrinter.cpp +++ b/lib/Target/MSP430/MSP430AsmPrinter.cpp @@ -134,8 +134,6 @@ bool MSP430AsmPrinter::runOnMachineFunction(MachineFunction &MF) { if (TAI->hasDotTypeDotSizeDirective()) O << "\t.size\t" << CurrentFnName << ", .-" << CurrentFnName << '\n'; - O.flush(); - // We didn't modify anything return false; } diff --git a/lib/Target/PowerPC/AsmPrinter/PPCAsmPrinter.cpp b/lib/Target/PowerPC/AsmPrinter/PPCAsmPrinter.cpp index 12a6da72818..3f1ac2cdc98 100644 --- a/lib/Target/PowerPC/AsmPrinter/PPCAsmPrinter.cpp +++ b/lib/Target/PowerPC/AsmPrinter/PPCAsmPrinter.cpp @@ -646,8 +646,6 @@ bool PPCLinuxAsmPrinter::runOnMachineFunction(MachineFunction &MF) { // Emit post-function debug information. DW->EndFunction(&MF); - O.flush(); - // We didn't modify anything. return false; } diff --git a/lib/Target/SystemZ/AsmPrinter/SystemZAsmPrinter.cpp b/lib/Target/SystemZ/AsmPrinter/SystemZAsmPrinter.cpp index ef64eb87cae..b82396895f9 100644 --- a/lib/Target/SystemZ/AsmPrinter/SystemZAsmPrinter.cpp +++ b/lib/Target/SystemZ/AsmPrinter/SystemZAsmPrinter.cpp @@ -144,8 +144,6 @@ bool SystemZAsmPrinter::runOnMachineFunction(MachineFunction &MF) { // Print out jump tables referenced by the function. EmitJumpTableInfo(MF.getJumpTableInfo(), MF); - O.flush(); - // We didn't modify anything return false; } diff --git a/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp b/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp index aea38cad030..2f56c2a9559 100644 --- a/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp +++ b/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp @@ -284,8 +284,6 @@ bool X86ATTAsmPrinter::runOnMachineFunction(MachineFunction &MF) { // Print out jump tables referenced by the function. EmitJumpTableInfo(MF.getJumpTableInfo(), MF); - O.flush(); - // We didn't modify anything. return false; } diff --git a/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp b/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp index 64037aec10a..c9a761d395c 100644 --- a/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp +++ b/lib/Target/X86/AsmPrinter/X86IntelAsmPrinter.cpp @@ -183,8 +183,6 @@ bool X86IntelAsmPrinter::runOnMachineFunction(MachineFunction &MF) { O << CurrentFnName << "\tendp\n"; - O.flush(); - // We didn't modify anything. return false; }