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

several additions to Rllvm #7

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

several additions to Rllvm #7

wants to merge 11 commits into from

Conversation

doktorschiwago
Copy link

Hi Duncan,

as written in my email I will start to send my changes upstream. This first set includes:

  1. The function PHIAddIncoming, that enables users to add source blocks to PHINodes
  2. The removal of the export of class functionType in NAMESPACE. With this export i couldnt run R CMD INSTALL on the package. The class FunctionType is already exported.
  3. The function finalizeEngine. According to http://blog.llvm.org/2013/07/using-mcjit-with-kaleidoscope-tutorial.html this function must be called before invoking code generated by mc jit.

I hope to find time in the next days to send the rest of the changes.

Chris

@duncantl
Copy link
Owner

Hi Christoph
Thanks for the pull request. I am very keen to integrate them.
I am just starting a very demanding week and I'll have very little time to do this right now.
If I haven't got to them by Monday 6th, please do ping me to remind me.

Much appreciated.
D.

runFunction is no longer supported
experiments/mcjit.R gives a primer on how to use mc jit
@doktorschiwago
Copy link
Author

No need to hurry! I just added some preliminary support for MC JIT in my
branch. Theres a small sample program in experiments that show how to
enable mc jit in llvm 3.5.

chris
Am 30.03.2015 03:34 schrieb "Duncan Temple Lang" [email protected]:

Hi Christoph
Thanks for the pull request. I am very keen to integrate them.
I am just starting a very demanding week and I'll have very little time to
do this right now.
If I haven't got to them by Monday 6th, please do ping me to remind me.

Much appreciated.
D.


Reply to this email directly or view it on GitHub
#7 (comment).

see experiments/DIBuilder.R in conjucntion with LLVM Kaleidoscope tutorial
for usage instructions
correcting a typo in definition of createPHI
external poiners, but objects of class Type
In llvm 3.6 DIBUILDER returns pointers, which should ease a non-leaking
implementation
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.

2 participants