This is an archive of the discontinued LLVM Phabricator instance.

[dsymutil] Rename llvm-dsymutil -> dsymutil
ClosedPublic

Authored by JDevlieghere on Mar 15 2018, 10:27 AM.

Details

Summary

Now that almost all functionality of Apple's dsymutil has been
upstreamed, the open source variant can be used as a drop in
replacement. Hence we feel it's no longer necessary to have the llvm
prefix.

Diff Detail

Repository
rL LLVM

Event Timeline

JDevlieghere created this revision.Mar 15 2018, 10:27 AM
aprantl accepted this revision.Mar 15 2018, 10:44 AM

Congratulations!
This is the kind of commit that is bound to break a lot of stuff, so please watch all the bots carefully.

This revision is now accepted and ready to land.Mar 15 2018, 10:44 AM
davide accepted this revision.Mar 15 2018, 10:46 AM

So excited to see this happening! :)

Have you checked whether the debuginfo-tests repository needs to be updated, too?

Congratulations!
This is the kind of commit that is bound to break a lot of stuff, so please watch all the bots carefully.

Thanks! Yup, good thing I'm Build Czar :-)

Have you checked whether the debuginfo-tests repository needs to be updated, too?

I haven't run them but I grepped for llvm-dsymutil in the mono-repo which includes debuginfo-tests. I guess that should be sufficient?

Yes that should be fine.

This revision was automatically updated to reflect the committed changes.