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

Brain - LLVM IR inline #7

Open
luizperes opened this issue Jul 4, 2016 · 8 comments
Open

Brain - LLVM IR inline #7

luizperes opened this issue Jul 4, 2016 · 8 comments

Comments

@luizperes
Copy link
Member

I was thinking of having a possibility to add LLVM IR.
That would be very useful for the creation of libraries for Brain. // using functions in the future
Any considerations?

@luizperes
Copy link
Member Author

luizperes commented Jul 4, 2016

@luizperes
Copy link
Member Author

This link here says that it is possible to link Brain with C files!!!

https://groups.google.com/forum/#!topic/llvm-dev/Cwq95-mqPys

@ryukinix
Copy link
Member

ryukinix commented Jul 6, 2016

Veeeeeeeeeeeeeeery nice! You'll attach this in Brain?

@rafaelcn
Copy link
Collaborator

rafaelcn commented Jul 6, 2016

Yes, he will :3

@luizperes
Copy link
Member Author

I will study this feature thouroughly, thinking better later, I'm re-organizing my thoughts together to implement LLVM IR, instead. This new approach came in my mind for two reasons.

  • First, that will take one layer off from the process of compiling.
  • Second, Brain will be compatible with any language that is LLVM-IR-based.

@rafaelcn rafaelcn added this to the 1.0 milestone Jul 12, 2016
@rafaelcn
Copy link
Collaborator

Well, I think we should review this issue, @luizperes has already implemented this great feature and now it provides us a great feature! 💯 🎉

@luizperes
Copy link
Member Author

It is prepared to be implemented, although it is not fully implemented. It works only for DebugExpr, InputExpr and OutputExpr so far. This feature is not going to be included on Brain 1.0, once we are running out of time.

Therefore I will reopen this issue and take it out from the Milestone 1.0 // that will be implemented if we have time

@luizperes luizperes reopened this Jul 29, 2016
@luizperes luizperes removed this from the 1.0 milestone Jul 29, 2016
@rafaelcn rafaelcn added this to the 1.1 milestone Sep 1, 2016
@luizperes luizperes removed this from the 1.1 milestone Dec 2, 2016
@luizperes
Copy link
Member Author

Also taken off milestone 1.1

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

3 participants