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

[Roland McGrath via Phabricator via llvm-commits]

mcgrathr added a comment.

This feature seems like a good addition. I'm wondering about the interactions with other switches and if it might make sense to rearchitect some of the CLI API here for the more arcane corners.  Maybe it's already orthogonal enough, but I'm not sure.  I'm thinking in particular about the various --strip-* options.  Perhaps it's the case that all such variations apply only to text IFS output?  I'm also wondering if there might be use cases for multiple different kinds of IFS output (i.e. different sets of stripping / arch options).  Come to think of it, we could even have uses for multiple ELF stub outputs from the same IFS input with different switches (e.g. arch).  So perhaps it makes sense to go in a direction where each output file is separately described with both its format (IFS vs ELF) and its format options (stripping, arch, etc), and there can be any number of such output files in whatever combination.  I'm not sure exactly what a CLI syntax for that would look like.  Another possibility that might fit well for build system integration is to accept a complex output specification in JSON either as a command-line switch value or from a file.

I suspect that even if we do want to pursue a fancier CLI API along those lines, this might be a reasonable incremental step in that direction.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115024