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

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 13:31:33 PDT 2016


> On Mar 25, 2016, at 1:30 PM, Sean Silva <chisophugis at gmail.com> wrote:
> 
> 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.

Like opt: only if you pass the input on stdin. If you give a filename it will produce a .ll derived from the input file name.

-- 
Mehdi



More information about the llvm-commits mailing list