This is an archive of the discontinued LLVM Phabricator instance.

llvm-nm and llvm-objcopy belong to toolchain
ClosedPublic

Authored by jiangyi on Apr 24 2019, 12:08 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

jiangyi created this revision.Apr 24 2019, 12:08 PM
This revision is now accepted and ready to land.Apr 25 2019, 11:59 PM

Ping. Does anyone have permission to commit this?

  1. patches which are meant to be sent for code review should include "full context" https://llvm.org/docs/Phabricator.html
  2. I think it'd be useful to flash out the description of this change (why you are changing this / what does this fix, even if the change is small)

my understanding is that you are trying to change LLVM_TOOLCHAIN_TOOLS - is it correct ?

  1. I will add one more reviewer to have a look at this diff (just in case).
compnerd accepted this revision.Jun 3 2019, 10:06 AM
compnerd closed this revision.Jun 3 2019, 10:14 AM

SVN r362425