[PATCH] D39821: Add CLANG_DEFAULT_OBJCOPY to allow Clang to use llvm-objcopy for dwarf fission

Jake Ehrlich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 10 10:36:13 PST 2017

jakehehrlich added a comment.

In https://reviews.llvm.org/D39821#921703, @Hahnfeld wrote:

> I think the current tests will only pass iff the executable name ends with `objcopy` - can we make this more reliable?

If I read the tests (split-debug.h, split-debug.c, and split-debug.s) correctly they check for the existence of the word "objcopy" somewhere in the executables name. I'm not sure how to make it anymore flexible than that. The right thing to do would be to somehow get the string from CMake and use that in this test. I'm not sure of llvm-lit supports any kind of find and replace like that however.



More information about the cfe-commits mailing list