[PATCH] D115024: [ifs][WIP] Patch llvm-ifs to allow output multiple types of stub file at the same time

Haowei Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 2 23:41:00 PST 2021


haowei created this revision.
haowei added reviewers: phosek, mcgrathr.
haowei requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

This is still a work in progress as I need to change all the unit tests files, but the implementation in llvm-ifs should be complete.

We are seeing use cases in outputting both ifs text stub and elf stub files from a single shared object in Fuchsia's build system. Instead of generating 2 llvm-ifs invocations, which will make llvm-ifs parse input ELF files twice, it is more efficient to just make llvm-ifs capable to generate every supported output format in a single run.

This patch adds "output-ifs", "output-elf" and "output-tbd" flags while still retain old "output-format", "output" flags just in case for compatibility. We can remove these old flags in a later revision.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D115024

Files:
  llvm/test/tools/llvm-ifs/output-target-error.test
  llvm/test/tools/llvm-ifs/write-stub.test
  llvm/tools/llvm-ifs/llvm-ifs.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D115024.391560.patch
Type: text/x-patch
Size: 16994 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211203/e97ccaf8/attachment.bin>


More information about the llvm-commits mailing list