Skip to content
This repository was archived by the owner on Mar 21, 2026. It is now read-only.

Regenerate lib/dynamic_clang.js for clang-9 - #18

Open
joelpurra wants to merge 6 commits into
tjfontaine:masterfrom
node-ffi-packager:llvm-clang-9
Open

Regenerate lib/dynamic_clang.js for clang-9#18
joelpurra wants to merge 6 commits into
tjfontaine:masterfrom
node-ffi-packager:llvm-clang-9

Conversation

@joelpurra

@joelpurra joelpurra commented Mar 7, 2020

Copy link
Copy Markdown

Forked and fixed quite a few more things: https://github.com/node-ffi-packager/node-libclang


Note: includes commits from previous pull-requests. Should be merged to master one by one, but wanted to showcase the stepwise progress and allow for feedback.


  • Uses clang-9 to regenerate support for itself.
  • Version 9 is the most recent major version, released 2019-09-19.
  • Has additional methods not supported by clang-6.0 (the only other version tested).
  • Dynamically targeting the right version might be possible, but also requires more effort in generation and tests.
  • Cross-generation between versions might be possible.
  • Built on Ubuntu 19.10 Eoan using from node-ffi-generate with suport for Elaborated types.
ln --symbolic --force "$(which "llvm-config-9")" "${HOME}/bin/llvm-config"
llvm-config --version
LD_LIBRARY_PATH="$(llvm-config --libdir)" node ./lib/generateffi.js

NOTE: move the generated newclang.js to replace lib/dynamic_clang.js.

See

> Represents a type that was referred to using an elaborated type keyword, e.g., `struct S`, or via a qualified name, e.g.,` N::M::type`, or both.

- Supporting Elaborated types allows for a greatly expanded library generation.
- Can be used by `ffi-generate` (with Elaborated type support) to regenerate the `lib/dynamic_clang.js`; see the `llvm-clang-6.0` branch.

See

- https://clang.llvm.org/doxygen/classclang_1_1ElaboratedType.html
- https://github.com/joelpurra/node-libclang/tree/llvm-clang-6.0
- https://github.com/joelpurra/node-ffi-generate/tree/support-elaborated-type
- Generated using `node-ffi-generate` with elaborated type support.
- Uses `clang-6.0`, which was released 2018-03-08.
- Has not been tested with older `clang`/`llvm` versions.
- Built on Ubuntu 19.10 Eoan using from `node-ffi-generate` with support for Elaborated types.

```shell
ln --symbolic --force "$(which "llvm-config-6.0")" "${HOME}/bin/llvm-config"
llvm-config --version
LD_LIBRARY_PATH="$(llvm-config --libdir)" node ./lib/generateffi.js
```

NOTE: move the generated `newclang.js` to replace `lib/dynamic_clang.js`.

See

- https://releases.llvm.org/6.0.0/docs/ReleaseNotes.html
- https://en.wikipedia.org/wiki/Clang
- https://github.com/tjfontaine/node-ffi-generate
- https://github.com/joelpurra/node-ffi-generate/tree/support-elaborated-type
- The code executes on `clang-6.0`.
- The fix makes it possible to use `clang-9`, the current major release.

See

- https://clang.llvm.org/
- Uses `clang-9` to regenerate support for itself.
- Version 9 is the most recent major version, released 2019-09-19.
- Has additional methods not supported by `clang-6.0` (the only other version tested).
- Dynamically targeting the right version might be possible, but also requires more effort in generation and tests.
- Cross-generation between versions might be possible.
- Built on Ubuntu 19.10 Eoan using from `node-ffi-generate` with support for Elaborated types.

```shell
ln --symbolic --force "$(which "llvm-config-9")" "${HOME}/bin/llvm-config"
llvm-config --version
LD_LIBRARY_PATH="$(llvm-config --libdir)" node ./lib/generateffi.js
```

NOTE: move the generated `newclang.js` to replace `lib/dynamic_clang.js`.

See

- https://releases.llvm.org/9.0.0/docs/ReleaseNotes.html
- https://en.wikipedia.org/wiki/Clang
- https://github.com/tjfontaine/node-ffi-generate
@joelpurra

Copy link
Copy Markdown
Author

Adds 200 or so lines of improvements between clang v6.0 and v9. In particular, I needed the new support for size_t to avoid some corruption issues when reading back struct data from a library using generated ref-struct parsing.

@joelpurra

Copy link
Copy Markdown
Author

Forked and fixed quite a few more things, might help someone: https://github.com/node-ffi-packager/node-libclang

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant