Page MenuHomePhabricator

llvm-nm and llvm-objcopy belong to toolchain
ClosedPublic

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

Details

Diff Detail

Repository
rL LLVM

Event Timeline

jiangyi created this revision.Apr 24 2019, 12:08 PM
Herald added a project: Restricted Project. · View Herald Transcript
alexshap accepted this revision.Apr 25 2019, 11:59 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.Mon, Jun 3, 10:06 AM
compnerd closed this revision.Mon, Jun 3, 10:14 AM

SVN r362425