[PATCH] D77231: [lld] Support -emit-asm with LTO

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 1 11:49:18 PDT 2020


MaskRay added a comment.

> Test Plan: ninja check-lld

This can be omitted as it does add value to the description. Almost every change will need `check-lld` (or `check-lld-elf` during development if you are confident non-ELF is not affected.)



================
Comment at: lld/test/ELF/lto/emit-asm.ll:3
+
+; RUN: ld.lld -m elf_x86_64 -lto-emit-asm -shared %t.o -o %t2.s
+; RUN: FileCheck --input-file %t2.s %s
----------------
Drop `-m elf_x86_64` and use `--lto-emit-asm`

`ld.lld --lto-emit-asm -shared %t.o -o - | FileCheck %s`


================
Comment at: lld/test/ELF/lto/emit-asm.ll:8
+; RUN:    -shared %t.o -o %t2.s
+; RUN: cat %t2.s %t2.s1 > %t3.s
+; RUN: FileCheck --input-file %t3.s %s
----------------
Check the contents of %t2.s and %t2.s1, respectively.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77231/new/

https://reviews.llvm.org/D77231





More information about the llvm-commits mailing list