[PATCH] D63341: [llvm-lipo] Implement -thin

Michael Trent via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 19 08:38:22 PDT 2019


mtrent added a comment.

In D63341#1547398 <https://reviews.llvm.org/D63341#1547398>, @alexshap wrote:

> i think that we should not "replicate cctools lipo error messages exactly", ours could be better / more convenient or informative where it makes sense.  
>  But in this particular case (see the comments above) I agree, including the names into the error message would be good!
>  Regarding exit codes  - yes, it's important (for many reasons) to make sure that our exit codes are correct.


I agree with all these points.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63341





More information about the llvm-commits mailing list