[PATCH] D26517: [LTO] Call llvm_shutdown on exit

[Davide Italiano via llvm-commits]

davide created this revision.
davide added reviewers: rafael, Bigcheese.
davide added a subscriber: llvm-commits.

On a non-lto build is a nop. On a LTO build, we deallocate/destroy managed static and this allows us to get the output of, e.g., -time-passes without performing a full shutdown


https://reviews.llvm.org/D26517

Files:
  ELF/Error.cpp
  test/ELF/lto/timepasses.ll


Index: test/ELF/lto/timepasses.ll
===================================================================
--- /dev/null
+++ test/ELF/lto/timepasses.ll
@@ -0,0 +1,13 @@
+; REQUIRES: x86
+; RUN: llvm-as %s -o %t.o
+; RUN: ld.lld -m elf_x86_64 %t.o -o %t.so -shared -mllvm -time-passes 2>&1 | FileCheck %s
+
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
+define void @patatino() {
+  ret void
+}
+
+; We should get the output of -time-passes even when --full-shutdown is not specified.
+; CHECK: Total Execution Time
Index: ELF/Error.cpp
===================================================================
--- ELF/Error.cpp
+++ ELF/Error.cpp
@@ -12,6 +12,7 @@
 
 #include "llvm/ADT/Twine.h"
 #include "llvm/Support/Error.h"
+#include "llvm/Support/ManagedStatic.h"
 #include "llvm/Support/raw_ostream.h"
 
 #if !defined(_MSC_VER) && !defined(__MINGW32__)
@@ -50,6 +51,12 @@
 void elf::exitLld(int Val) {
   outs().flush();
   errs().flush();
+
+  // Dealloc/destroy ManagedStatic variables before calling
+  // _exit(). In a non-LTO build, this is a nop. In an LTO
+  // build allows us to get the output of -time-passes.
+  llvm_shutdown();
+
   _exit(Val);
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26517.77523.patch
Type: text/x-patch
Size: 1218 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161110/25d2ba67/attachment.bin>