v8: add additional include dirs on AIX #289
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
On AIX, we now include
src/wasm/float16.h
from withinsrc/utils/utils.h
andsrc/wasm/float16.h
includes additional header files.For reference this patch added the changes.
Richard ran a test build last week (updated to V8 13.0.177) and it failed with:
We need to update the include dirs on AIX/IBM i to find these headers.
Update Test Build: https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix72-ppc64/53642/console
CC
@richardlau
@targos