[PATCH] D18415: [LTO] Basic support for internalize.

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 13:30:17 PDT 2016


silvas added inline comments.

================
Comment at: test/ELF/lto/internalize-basic.ll:4
@@ +3,3 @@
+; RUN: ld.lld -m elf_x86_64 %t.o -o %t2 -save-temps
+; RUN: llvm-dis %t2.lto.bc -o - | FileCheck %s
+
----------------
davide wrote:
> grimar wrote:
> > "-o -" ?  That looks a bit wierd to have "-" as a file name IMO, may be just standart temporary file would be better here ?
> It's stdout.
Doesn't llvm-dis default to stdout? I don't think `-o -` os needed.


http://reviews.llvm.org/D18415





More information about the llvm-commits mailing list