[PATCH] D107593: [Lanai] fix MC / objdump

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 9 09:58:25 PDT 2021


MaskRay added a comment.

You may remove `test` from the filename. The leading `test` path component says these are tests.



================
Comment at: llvm/test/tools/llvm-objdump/ELF/Lanai/smoketest.ll:1
+; RUN: llc  -o %t.o -filetype=obj -mtriple=lanai %s
+; RUN: llvm-objdump  -d -S %t.o | FileCheck %s
----------------
delete excess spaces between `llc` and `-o`

I usually prefer `%t.bc` for bitcode output


================
Comment at: llvm/test/tools/llvm-objdump/ELF/Lanai/smoketest.ll:2
+; RUN: llc  -o %t.o -filetype=obj -mtriple=lanai %s
+; RUN: llvm-objdump  -d -S %t.o | FileCheck %s
+
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107593



More information about the llvm-commits mailing list