[PATCH] D60924: [ThinLTO] Adding architecture name into saved object filename

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 19 14:56:23 PDT 2019


dexonsmith added a comment.

In D60924#1473161 <https://reviews.llvm.org/D60924#1473161>, @steven_wu wrote:

> In D60924#1473135 <https://reviews.llvm.org/D60924#1473135>, @dexonsmith wrote:
>
> > Is there a good way to test this?
>
>
> Not currently in LLVM. It might be possible to add a test in clang repo which requires DARWIN and ld64.


Maybe we can split out the function to generate the filename, and add a unit test for that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60924





More information about the llvm-commits mailing list