Skip to content

Conversation

@ccurtsinger
Copy link
Contributor

LLVM uses standard line info opcodes from DWARF 3/4, but specifies version 2.

@ccurtsinger
Copy link
Contributor Author

I just pushed changes to master that drop the standard_opcodes handling. If an opcode is less than opcode_base, it is handled with the standard meaning in DWARF 4. Other opcodes are handled as vendor-specific.

I'll submit the uninitialized null pointer fix as a separate pull request.

@aclements
Copy link
Owner

Great.

When you resubmit the null pointer fix, would you mind doing it in the constructor initializer list? Currently, libelfin compiles with GCC 4.6, and as nice as member initializers are, I'd like to keep 4.6 compatibility until it dies off a little more (or until there's a really good reason to drop 4.6 support).

@aclements
Copy link
Owner

I've cherry-picked (and tweaked) most of these. I left out the shared libraries ones because it looked like you had undone those anyway. Feel free to open another PR with a commit to add shared library support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants