std.Build: add {make, configure} phase field to Build #17639
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
help detect possibly hidden dependencies on the running order of steps, especially in -j1 mode
* support 0x prefixed hex code for CLI seed arguments * don't change the build summary; the printed CLI on build runner failure is sufficient * use `std.crypto.random` instead of system time for entropy
…ep creation is in configure phase and step's make runs in make phase
StringIndexAdapter.bytes Without this change, constructing a Context/Adapter for a `const` string table requires a `@constCast`.
std.Build: add --seed argument to randomize step dependencies spawning
elf: actually check for dynamic executables
This reverts commit 78855bd. This commit did not replace uses of `Type.err_int` of which there are currently 60 uses. Re-opens ziglang#786
There is this `common` prefix which was not getting passed in all union tags of pointer vals, resumably making InternPool slower for pointer values.
Currently, Zig uses the oldest Debian release that is still under LTS for the default version minimum. This is now Debian 10 (Buster), with long-term support until 2024-06-30. Debian Buster uses Linux 4.19 and glibc 2.28. For the default version maximum, Zig uses the newest stable Linux version, which is currently 6.5.7. Citations: * https://www.debian.org/News/2019/20190706 * https://packages.debian.org/source/buster/glibc * https://kernel.org/ Previous update commit: 1530203
I observed some out-of-memory conditions happening on one of the CI servers. This should make it avoid scheduling compiler builds at the same time as other memory-intensive operations.
Previous commit caused this error to be printed: zig build-exe zig Debug aarch64-macos-none: error: memory usage peaked at 6323765248 bytes, exceeding the declared upper bound of 5200000000
Looks like the higher memory requirements come from building with a clean cache, and affect all operating systems, not just macOS.
zig build-exe zig Debug aarch64-macos-none: error: memory usage peaked at 6466420736 bytes, exceeding the declared upper bound of 6350000000
zig build-exe zig Debug aarch64-macos-none: error: memory usage peaked at 6607110144 bytes, exceeding the declared upper bound of 6500000000
…#17617) RSA exponents are typically 3 or 65537, and public. For those, we don't need to use conditional moves on the exponent, and precomputing a lookup table is not worth it. So, save a few cpu cycles and some memory for that common case. For safety, make `powWithEncodedExponent()` constant-time by default, and introduce a `powWithEncodedPublicExponent()` function for exponents that are assumed to be public. With `powWithEncodedPublicExponent()`, short (<= 36 bits) exponents will take the fast path.
Tests with +-0, numbers with accuracy 0.001, 0.000001, +-max for inf.
std.rand: Support integers with >64 bits in more functions
dwarf: decouple DWARF formats from target pointer size + misc fixes
Fixes ziglang#15038. The goal here is to guarantee lossyCast() is panic-free and always safe.
* Add missing period in Stack's description This looks fine in the source, but looks bad when seen on the documentation website. * Correct documentation for attachSegfaultHandler() The description for attachSegfaultHandler() looks pretty bad without indicating that the stuff at the end is code * Added missing 'the's in Queue.put's documentation * Fixed several errors in Stack's documentation `push()` and `pop()` were not styled as code There was no period after `pop()`, which looks bad on the documentation. * Fix multiple problems in base64.zig Both "invalid"s in Base64.decoder were not capitalized. Missing period in documentation of Base64DecoderWithIgnore.calcSizeUpperBound. * Fix capitalization typos in bit_set.zig In DynamicBitSetUnmanaged.deinit's and DynamicBitSet.deinit's documentation, "deinitializes" was uncapitalized. * Fix typos in fifo.zig's documentation Added a previously missing period to the end of the first line of LinearFifo.writableSlice's documentation. Added missing periods to both lines of LinearFifo.pump's documentation. * Fix typos in fmt.bufPrint's documentation The starts of both lines were not capitalized. * Fix minor documentation problems in fs/file.zig Missing periods in documentation for Permissions.setReadOnly, PermissionsWindows.setReadOnly, MetadataUnix.created, MetadataLinux.created, and MetadataWindows.created. * Fix a glaring typo in enums.zig * Correct errors in fs.zig * Fixed documentation problems in hash_map.zig The added empty line in verify_context's documentation is needed, otherwise autodoc for some reason assumes that the list hasn't been terminated and continues reading off the rest of the documentation as if it were part of the second list item. * Added lines between consecutive URLs in http.zig Makes the documentation conform closer to what was intended. * Fix wrongfully ended sentence in Uri.zig * Handle wrongly entered comma in valgrind.zig. * Add missing periods in wasm.zig's documentation * Fix odd spacing in event/loop.zig * Add missing period in http/Headers.zig * Added missing period in io/limited_reader.zig This isn't in the documentation due to what I guess is a limitation of autodoc, but it's clearly supposed to be. If it was, it would look pretty bad. * Correct documentation in math/big/int.zig * Correct formatting in math/big/rational.zig * Create an actual link to ZIGNOR's paper. * Fixed grammatical issues in sort/block.zig This will not show up in the documentation currently. * Fix typo in hash_map.zig
…programming/zig into make-configure-phase
|
Closing abandoned pull request: failing CI checks, merge conflicts, 30+ days old. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
assert that step creation is in configure phase and step's make runs in make phase
This is an attempt to resolve #14941