[PATCH] D33513: [ThinLTO] Fix ThinLTO crash while destroying context

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 29 10:25:15 PDT 2017


mehdi_amini added inline comments.


================
Comment at: test/ThinLTO/X86/lazy-load-temporaries-cleanup.ll:4
+; RUN: opt -module-summary -bitcode-mdindex-threshold=0 %S/Inputs/lazy-load-temporaries-cleanup.b.ll -o %t.2.o
+; RUN: llvm-dis -materialize-metadata %t.2.o
----------------
Just a nit: usually we don't split the RUN commands using a temporary file unless needed (i.e. use a pipe, opt can print on stdout and llvm-dis reads on stdin).


https://reviews.llvm.org/D33513





More information about the llvm-commits mailing list