[PATCH] D110527: [llvm-reduce] Add MIR support.

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

I'm looking at using this functionality, and ran into a question on the design.  I'm not clear why you exposed an mtriple command line argument on llvm-reduce.  It appears you're using this to derive a default data layout, and as an input to parsing the machine function.  However, LLVM IR allows specifying both data layout and triple in IR.  As a result, we should be able to have the parsing derive the appropriate target directly from the test case.  Is there something I'm missing here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110527