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

Basic support for extracting values of global variables #23

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

mkmks
Copy link
Contributor

@mkmks mkmks commented Oct 17, 2011

No description provided.

@bos
Copy link
Owner

bos commented May 17, 2012

This doesn't build against LLVM 3.0 because you edited the configure.ac script.

Also, this warning makes me nervous:

LLVM/Core/Instructions.hs:216:1:
    Warning: Pattern match(es) are non-exhaustive
             In an equation for `getValConvArg':
                 Patterns not matched:
                     IDRet _ _
                     IDRetVoid
                     IDBrCond _ _ _
                     IDBrUncond _
                     ...

@mkmks mkmks closed this Mar 31, 2013
@mkmks mkmks reopened this Mar 31, 2013
@mkmks
Copy link
Contributor Author

mkmks commented Mar 31, 2013

I've checked the build against LLVM 3.2 and fixed that warning.

@mkmks
Copy link
Contributor Author

mkmks commented Mar 31, 2013

Also closes #37.

@mkmks
Copy link
Contributor Author

mkmks commented Mar 31, 2013

And #43.

@NathanHowell
Copy link
Collaborator

Commits like f7e3108 churn a huge amount of code due to formatting changes. Could you rebase this pull request down to a smaller number of commits against the HEAD and avoid unnecessary formatting changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants