Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Edit CMake and remove unecessary dependencies #8

Open
maxgillett opened this issue Apr 9, 2023 · 2 comments
Open

Edit CMake and remove unecessary dependencies #8

maxgillett opened this issue Apr 9, 2023 · 2 comments
Labels
good first issue Good for newcomers

Comments

@maxgillett
Copy link
Member

See https://github.com/tonlabs/TON-Compiler repo structure for an example of how to remove everything not required for backend compilation.

@maxgillett maxgillett added the good first issue Good for newcomers label Apr 9, 2023
@vivekvpandya
Copy link
Collaborator

The linked code seems to be using old LLVM repo. So if you would like to have regular updates from LLVM community and if up streaming this work to LLVM community is desired then its not good idea to make things as per above project.
See https://llvm.org/docs/Proposals/GitHubMove.html

Correct me if this understanding is wrong.

@maxgillett
Copy link
Member Author

Ah, I'm not suggesting that we change the directory structure, just that this repo should be a standalone product and only contain dependencies required to build the compiler. I'm not advocating for making any changes to the target independent code unless they're absolutely necessary. It's definitely a goal to one day upstream the backend target to the llvm-project repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants