search

digego / extempore

A cyber-physical programming environment
1.41k stars 127 forks source link

Extract llvm_zone_t code into its own namespace. #398

Closed nic-donaldson closed 3 years ago

nic-donaldson commented 3 years ago

Following a similar PR for closure_address_table, this PR moves llvm_zone_t into its own space.

This PR is carved out from this larger diff: https://github.com/digego/extempore/compare/master...nic-donaldson:llvm-refactor#files_bucket which upgrades to LLVM 11. This is part of separating code that includes or links with LLVM code from code that doesn't.

I made a couple of small changes in some functions while doing this, I'll call out those spots with comments because it's hard to spot code that has moved + changed.

benswift commented 3 years ago

Hey @nic-donaldson are you able to rebase this to the current master (55b47d7deff2cb2e313cbcfaa3e0a085fd400f73) and force push (which should update this PR)? That will re-run the tests (inc. on macOS 11) as well.

nic-donaldson commented 3 years ago

Hey @nic-donaldson are you able to rebase this to the current master (55b47d7) and force push (which should update this PR)? That will re-run the tests (inc. on macOS 11) as well.

yep, done! let's hope they pass :|