Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cargo-objdump, cargo-size, etc. #51

Closed
1 task done
japaric opened this issue Feb 23, 2018 · 8 comments
Closed
1 task done

cargo-objdump, cargo-size, etc. #51

japaric opened this issue Feb 23, 2018 · 8 comments

Comments

@japaric
Copy link
Member

japaric commented Feb 23, 2018

Triage

2018-06-29

The llvm-tools rustup component now exists and cargo-binutils is making use of it. Check this announcement for details.

Let's discuss the final UI we want for the Cargo subcommands in the cargo-binutils issue tracker.


Once #50 is accepted / implemented it would be nice to have wrapper over e.g. llvm-objdump for
convenience. The wrapper would take care of passing -triple=$target to llvm-objdump, demangling
the output, passing the path to the binary to llvm-objdump, etc.

We don't have to wait until #50 is implemented though; we can start prototyping on top of
arm-none-eabi-objdump.

Tasks

Originally discussed in #43 but I think this more in line with #44

cc @thejpster

@japaric
Copy link
Member Author

japaric commented Feb 28, 2018

Adding to the epoch 2018 milestone as discussed in yesterday's meeting

@japaric japaric added this to the Epoch 2018 milestone Feb 28, 2018
@dvc94ch
Copy link
Member

dvc94ch commented Mar 30, 2018

Can we also add cargo-dwarfdump [0]

[0] https://llvm.org/docs/CommandGuide/llvm-dwarfdump.html

@whitequark
Copy link

When was the last time you needed dwarfdump and were not developing an LLVM backend?..

@grossws
Copy link

grossws commented Apr 4, 2018

@m4b wrote cargo-sym which can be used as limited objdump replacement.

@japaric
Copy link
Member Author

japaric commented Apr 4, 2018

@dvc94ch I didn't include that tool in the discussion issue I opened (rust-lang/rust#49584). You can bring it up in the comments.

During yesterday's meeting @jcsoo brought up the idea of having a step 0 for these subcommands: a minimal implementation could simply proxy the LLVM tool; for example, cargo nm -- -C path/to/binary would be sugar for $(rustc --print sysroot)/path/to/llvm-nm -C path/to/binary.

@whitequark
Copy link

@m4b wrote cargo-sym which can be used as limited objdump replacement.

Please, no. This would mean that e.g. the OpenRISC LLVM/rustc port that I maintain would be gated on Capstone gaining OpenRISC support before standard rustc tools could be used.

@japaric
Copy link
Member Author

japaric commented Apr 7, 2018

During yesterday's meeting @jcsoo brought up the idea of having a step 0 for these subcommands: a minimal implementation could simply proxy the LLVM tool

I have implemented step 0 in https://github.com/japaric/cargo-binutils

@japaric
Copy link
Member Author

japaric commented Jun 29, 2018

The llvm-tools rustup component now exists and cargo-binutils is making use of it. Check this announcement for details.

Let's discuss the final UI we want for the Cargo subcommands in the cargo-binutils issue tracker.

@japaric japaric closed this as completed Jun 29, 2018
@japaric japaric mentioned this issue Jul 3, 2018
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants