-
Notifications
You must be signed in to change notification settings - Fork 12.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rollup of 9 pull requests #128883
Rollup of 9 pull requests #128883
Commits on Aug 8, 2024
-
Configuration menu - View commit details
-
Copy full SHA for fb1dac2 - Browse repository at this point
Copy the full SHA fb1dac2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1e3ea29 - Browse repository at this point
Copy the full SHA 1e3ea29View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2abcc6b - Browse repository at this point
Copy the full SHA 2abcc6bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 09ae438 - Browse repository at this point
Copy the full SHA 09ae438View commit details -
Configuration menu - View commit details
-
Copy full SHA for c966370 - Browse repository at this point
Copy the full SHA c966370View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7c4150f - Browse repository at this point
Copy the full SHA 7c4150fView commit details
Commits on Aug 9, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 11b801b - Browse repository at this point
Copy the full SHA 11b801bView commit details -
Configuration menu - View commit details
-
Copy full SHA for a3f8edf - Browse repository at this point
Copy the full SHA a3f8edfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 625432c - Browse repository at this point
Copy the full SHA 625432cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0106f5b - Browse repository at this point
Copy the full SHA 0106f5bView commit details -
VxWorks: Add safety comment for vxCpuEnabledGet
Co-authored-by: Trevor Gross <t.gross35@gmail.com>
Configuration menu - View commit details
-
Copy full SHA for b20d9f0 - Browse repository at this point
Copy the full SHA b20d9f0View commit details -
tests: add regression test for incorrect
BytePos
manipulation trigg……ering assertion Issue: <rust-lang#128717>
Configuration menu - View commit details
-
Copy full SHA for b589f86 - Browse repository at this point
Copy the full SHA b589f86View commit details -
hir_typeck: use
end_point
overBytePos
manipulationsParser has error recovery for Unicode-confusables, which includes the right parentheses `)`. If a multi-byte right parentheses look-alike reaches the argument removal suggestion diagnostics, it would trigger an assertion because the diagnostics used `- BytePos(1)` which can land within a multi-byte codepoint. This is fixed by using `SourceMap::end_point` to find the final right delimiter codepoint, which correctly respects codepoint boundaries.
Configuration menu - View commit details
-
Copy full SHA for 879bfd7 - Browse repository at this point
Copy the full SHA 879bfd7View commit details -
tests: add regression test for rust-lang#128845
For codepoint boundary assertion triggered by a let stmt compound assignment removal suggestion when encountering recovered multi-byte compound ops. Issue: <rust-lang#128845>
Configuration menu - View commit details
-
Copy full SHA for 92520a9 - Browse repository at this point
Copy the full SHA 92520a9View commit details -
parser: ensure let stmt compound assignment removal suggestion respec…
…t codepoint boundaries Previously we would try to issue a suggestion for `let x <op>= 1`, i.e. a compound assignment within a `let` binding, to remove the `<op>`. The suggestion code unfortunately incorrectly assumed that the `<op>` is an exactly-1-byte ASCII character, but this assumption is incorrect because we also recover Unicode-confusables like `➖=` as `-=`. In this example, the suggestion code used a `+ BytePos(1)` to calculate the span of the `<op>` codepoint that looks like `-` but the mult-byte Unicode look-alike would cause the suggested removal span to be inside a multi-byte codepoint boundary, triggering a codepoint boundary assertion. Issue: <rust-lang#128845>
Configuration menu - View commit details
-
Copy full SHA for d65f131 - Browse repository at this point
Copy the full SHA d65f131View commit details -
Configuration menu - View commit details
-
Copy full SHA for 38874a6 - Browse repository at this point
Copy the full SHA 38874a6View commit details -
add
builder-config
into tarball sourcesThis will be useful for certain scenarios where developers want to know how the tarball sources were generated. We also want this to check for CI rustc incompatible options on bootstrap. Signed-off-by: onur-ozkan <work@onurozkan.dev>
Configuration menu - View commit details
-
Copy full SHA for a7c415c - Browse repository at this point
Copy the full SHA a7c415cView commit details -
use absolute path for
config.toml
Signed-off-by: onur-ozkan <work@onurozkan.dev>
Configuration menu - View commit details
-
Copy full SHA for ff0d37c - Browse repository at this point
Copy the full SHA ff0d37cView commit details -
Configuration menu - View commit details
-
Copy full SHA for f72cb04 - Browse repository at this point
Copy the full SHA f72cb04View commit details -
Configuration menu - View commit details
-
Copy full SHA for a380d5e - Browse repository at this point
Copy the full SHA a380d5eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1d19c2c - Browse repository at this point
Copy the full SHA 1d19c2cView commit details -
Rollup merge of rust-lang#128815 - Nadrieril:is_stolen, r=jieyouxu,lcnr
Add `Steal::is_stolen()` Writers of rustc drivers (such as myself) often encounter stealing issues. It is currently impossible to gracefully handle them. This PR adds a `Steal::is_stolen()` function for that purpose.
Configuration menu - View commit details
-
Copy full SHA for 55329cc - Browse repository at this point
Copy the full SHA 55329ccView commit details -
Rollup merge of rust-lang#128817 - biabbas:vxworks_update, r=tgross35
VxWorks code refactored 1. Extern TaskNameSet as minimum supported version of os is VxWorks 7 which would have taskNameSet 2. Vx_TASK_NAME_LEN is 31 on VxWorks7, defined variable res. 3. Add unsafe blocks on Non::Zero usage in available_parallelism() 4. Update vxworks docs. r? `@tgross35` cc `@devnexen`
Configuration menu - View commit details
-
Copy full SHA for 5372936 - Browse repository at this point
Copy the full SHA 5372936View commit details -
Rollup merge of rust-lang#128822 - onur-ozkan:add-build-config-in-tar…
…balls, r=Kobzol add `builder-config` into tarball sources This will be useful for certain scenarios where developers want to know how the tarball sources were generated. We also want this to check for CI rustc incompatible options on bootstrap. Blocker for rust-lang#122709 r? Kobzol
Configuration menu - View commit details
-
Copy full SHA for 5f1e25e - Browse repository at this point
Copy the full SHA 5f1e25eView commit details -
Rollup merge of rust-lang#128838 - notriddle:notriddle/invalid-tag-is…
…-not-rust, r=GuillaumeGomez rustdoc: do not run doctests with invalid langstrings rust-lang#124577 (comment) CC ``@decathorpe``
Configuration menu - View commit details
-
Copy full SHA for 9f11eb1 - Browse repository at this point
Copy the full SHA 9f11eb1View commit details -
Rollup merge of rust-lang#128852 - folkertdev:multipart-suggestion-st…
…able-sort, r=compiler-errors use stable sort to sort multipart diagnostics I think a stable sort should be used to sort the different parts of a multipart selection. The current unstable sort uses the text of the suggestion as a tie-breaker. That just doesn't seem right, and the order of the input is a better choice I think, because it gives the diagnostic author more control. This came up when I was building a suggestion where ```rust fn foo() {} ``` must be turned into an unsafe function, and an attribute must be added ```rust #[target_feature(enable = "...")] unsafe fn foo() {} ``` In this example, the two suggestions occur at the same position, but the order is extremely important: unsafe must come after the attribute. But the situation changes if there is a pub/pub(crate), and if the unsafe is already present. It just out that because of the suggestion text, there is no way for me to order the suggestions correctly. This change probably should be tested, but are there tests of the diagnostics code itself in the tests? r? ```@estebank```
Configuration menu - View commit details
-
Copy full SHA for 2066002 - Browse repository at this point
Copy the full SHA 2066002View commit details -
Rollup merge of rust-lang#128859 - MinxuanZ:mips-sig, r=Amanieu
Fix the name of signal 19 in library/std/src/sys/pal/unix/process/process_unix/tests.rs for mips/sparc linux relate to rust-lang#128816
Configuration menu - View commit details
-
Copy full SHA for 2e0c5ad - Browse repository at this point
Copy the full SHA 2e0c5adView commit details -
Rollup merge of rust-lang#128864 - jieyouxu:funnicode, r=Urgau
Use `SourceMap::end_point` instead of `- BytePos(1)` in arg removal suggestion Previously, we tried to remove extra arg commas when providing extra arg removal suggestions. One of the edge cases is having to account for an arg that has a closing delimiter `)` following it. However, the previous suggestion code assumed that the delimiter is in fact exactly the 1-byte `)` character. This assumption was proven incorrect, because we recover from Unicode-confusable delimiters in the parser, which means that the ending delimiter could be a multi-byte codepoint that looks *like* a `)`. Subtracing 1 byte could land us in the middle of a codepoint, triggering a codepoint boundary assertion. This is fixed by using `SourceMap::end_point` which properly accounts for codepoint boundaries. Fixes rust-lang#128717. cc ````@fmease```` and rust-lang#128790
Configuration menu - View commit details
-
Copy full SHA for 9eb77ac - Browse repository at this point
Copy the full SHA 9eb77acView commit details -
Rollup merge of rust-lang#128865 - jieyouxu:unicurd, r=Urgau
Ensure let stmt compound assignment removal suggestion respect codepoint boundaries Previously we would try to issue a suggestion for `let x <op>= 1`, i.e. a compound assignment within a `let` binding, to remove the `<op>`. The suggestion code unfortunately incorrectly assumed that the `<op>` is an exactly-1-byte ASCII character, but this assumption is incorrect because we also recover Unicode-confusables like `➖=` as `-=`. In this example, the suggestion code used a `+ BytePos(1)` to calculate the span of the `<op>` codepoint that looks like `-` but the mult-byte Unicode look-alike would cause the suggested removal span to be inside a multi-byte codepoint boundary, triggering a codepoint boundary assertion. The fix is to use `SourceMap::start_point(token_span)` which properly accounts for codepoint boundaries. Fixes rust-lang#128845. cc rust-lang#128790 r? ````@fmease````
Configuration menu - View commit details
-
Copy full SHA for 665a1a4 - Browse repository at this point
Copy the full SHA 665a1a4View commit details -
Rollup merge of rust-lang#128874 - Kobzol:cmd-verbose-logging, r=onur…
…-ozkan Disable verbose bootstrap command failure logging by default One of my recent bootstrap command refactoring PRs enabled verbose logging of command failures by default. While this is great for debugging bootstrap, in many situations it's just too verbose and prevents the user from seeing the actual printed stdout/stderr, which usually contains much more useful information. This PR reverts that logic, and only prints a detailed error when `-v` is passed to bootstrap. r? ````@onur-ozkan````
Configuration menu - View commit details
-
Copy full SHA for cea3b42 - Browse repository at this point
Copy the full SHA cea3b42View commit details