-
Notifications
You must be signed in to change notification settings - Fork 206
Fake PR to test the upstreaming of commit up to gerris/rebase-upstream/2026-01-10 #4373
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
Open
dkm
wants to merge
22
commits into
dkm/upstream-base
Choose a base branch
from
gerris/rebase-upstream/2026-01-10
base: dkm/upstream-base
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or 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
gcc/rust/ChangeLog: * resolve/rust-early-name-resolver-2.0.cc (Early::visit_attributes): Mention the name of the attribute macro that hasn't been found.
This patch refactors all uses of vector patterns since a vector of patterns would be considered as an alt pattern, a vector is considered useless. gcc/rust/ChangeLog: * ast/rust-ast-builder.cc (Builder::match_arm): Moves the vector of patterns to a single pattern. * ast/rust-ast-collector.cc (TokenCollector::visit):Likewise. * ast/rust-ast-pointer-visitor.cc (PointerVisitor::visit):Likewise. * ast/rust-ast-visitor.cc (DefaultASTVisitor::visit):Likewise. * ast/rust-ast.cc (IfLetExpr::as_string):Likewise. (WhileLetLoopExpr::as_string):Likewise. (MatchArm::as_string):Likewise. * ast/rust-desugar-question-mark.cc (make_match_arm):Likewise. * ast/rust-desugar-while-let.cc (DesugarWhileLet::desugar):Likewise. * ast/rust-expr.h (class WhileLetLoopExpr):Likewise. (class IfLetExpr):Likewise. * backend/rust-compile-expr.cc (CompileExpr::visit):Likewise. * checks/errors/rust-hir-pattern-analysis.cc (lower_arm):Likewise. * expand/rust-cfg-strip.cc (CfgStrip::visit):Likewise. * hir/rust-ast-lower.cc (ASTLoweringIfLetBlock::desugar_iflet):Likewise. (ASTLoweringExprWithBlock::visit):Likewise. * hir/rust-hir-dump.cc (Dump::do_matcharm):Likewise. (Dump::visit):Likewise. * hir/tree/rust-hir-expr.cc (OperatorExpr::operator=):Likewise. (ArithmeticOrLogicalExpr::operator=):Likewise. (ComparisonExpr::operator=):Likewise. (LazyBooleanExpr::operator=):Likewise. (TypeCastExpr::operator=):Likewise. (AssignmentExpr::operator=):Likewise. (CompoundAssignmentExpr::operator=):Likewise. (GroupedExpr::operator=):Likewise. (ArrayExpr::operator=):Likewise. (ArrayIndexExpr::operator=):Likewise. (CallExpr::operator=):Likewise. (MethodCallExpr::operator=):Likewise. (FieldAccessExpr::operator=):Likewise. (BlockExpr::operator=):Likewise. (BreakExpr::operator=):Likewise. (ReturnExpr::operator=):Likewise. (UnsafeBlockExpr::operator=):Likewise. (BaseLoopExpr::operator=):Likewise. (WhileLetLoopExpr::WhileLetLoopExpr):Likewise. (WhileLetLoopExpr::operator=):Likewise. (MatchArm::MatchArm):Likewise. (MatchArm::operator=):Likewise. (MatchExpr::operator=):Likewise. * hir/tree/rust-hir-expr.h (class WhileLetLoopExpr):Likewise. * hir/tree/rust-hir-visitor.cc (DefaultHIRVisitor::walk):Likewise. (DefaultHIRVisitor::visit_match_arm):Likewise. * hir/tree/rust-hir.cc (WhileLetLoopExpr::as_string):Likewise. (MatchArm::as_string):Likewise. * parse/rust-parse-impl-expr.hxx: Likewise. * parse/rust-parse-impl.hxx: Likewise. * parse/rust-parse.h:Likewise. * resolve/rust-default-resolver.cc (DefaultResolver::visit_if_let_patterns):Likewise. * resolve/rust-late-name-resolver-2.0.cc (Late::visit):Likewise. * typecheck/rust-hir-type-check-expr.cc (TypeCheckExpr::visit):Likewise. Signed-off-by: Lucas Ly Ba <[email protected]>
When a macro expansion fails (e.g. due to a parsing error like invalid syntax in the macro body), the expander previously returned an error fragment but did not update the AST. This left the original macro invocation in place, which subsequently caused an ICE (rogue macro detected) during the lowering phase. This patch updates `expand_invoc` to replace the macro invocation with an empty fragment if expansion fails, ensuring the compiler can proceed (or exit gracefully) without crashing. Fixes #4213 gcc/rust/ChangeLog: * expand/rust-macro-expand.cc (MacroExpander::expand_invoc): Handle error fragments by replacing them with empty fragments. gcc/testsuite/ChangeLog: * rust/compile/issue-4213.rs: New test. Signed-off-by: Harishankar <[email protected]>
A parser error header with Parse::Error namespace has recently been introduced. Move some old parser error classes to this namespace. gcc/rust/ChangeLog: * parse/rust-parse.h (class ParseLifetimeParamError): Move error from here ... (class ParseLifetimeError): Likewise. (enum class): Likewise. * parse/rust-parse-error.h (class LifetimeParam): ... to here. here. (class Lifetime): Likewise. (enum class): Likewise. (struct LoopLabel): Likewise and make it a full struct with ctors. (struct Self): Likewise. * parse/rust-parse-impl-expr.hxx: Make error point to new namespace. * parse/rust-parse-impl.hxx: Likewise. Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
Errors in the parser needs to be collected instead of emitted directly. gcc/rust/ChangeLog: * parse/rust-parse-impl.hxx: Collect errors instead of emitting them. Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
gcc/rust/ChangeLog: * parse/rust-parse-error.h (struct BlockExpr): Add BlockExpr error type * parse/rust-parse-impl-expr.hxx: Update return types. * parse/rust-parse-impl.hxx: Likewise. * parse/rust-parse.h: Update function prototypes. Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
Some error types in the parser had the exact same meaning and could be grouped under one same type. gcc/rust/ChangeLog: * ast/rust-ast.cc (AttributeParser::parse_meta_item_lit): Use tl::expected * expand/rust-macro-builtins-helpers.cc (parse_single_string_literal): Likewise. * expand/rust-macro-expand.cc (MacroExpander::match_fragment): Likewise. * parse/rust-cfg-parser.cc (parse_cfg_option): Likewise. * parse/rust-parse-error.h (struct SimplePath): Remove error type. (struct DelimTokenTree): Likewise. (struct Token): Likewise. (struct TokenTree): Likewise. (class LifetimeParam): Move from here ... (struct LifetimeParam): ... to here. Add ctor. (class Lifetime): Add error type. (enum class): Remove AnonConst. (struct BlockExpr): Change BlockExpr to generic node error. * parse/rust-parse-impl-expr.hxx: Use tl::expected for errors. * parse/rust-parse-impl-path.hxx: Likewise. * parse/rust-parse-impl-ttree.hxx: Likewise. * parse/rust-parse-impl.hxx: Likewise. * parse/rust-parse.h: Update function return types with tl::expected to propagate errors. Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
gcc/rust/ChangeLog: * ast/rust-ast-collector.cc (TokenCollector::visit): Update function name. * ast/rust-ast-pointer-visitor.cc (PointerVisitor::visit): Likewise. * ast/rust-ast-visitor.cc (DefaultASTVisitor::visit): Likewise. * ast/rust-ast.cc (BreakExpr::as_string): Use getter function. (AttributeParser::parse_path_meta_item): Convert to expected type. (ReturnExpr::as_string): Change access to returned expr. * ast/rust-desugar-for-loops.cc (DesugarForLoops::DesugarCtx::make_break_arm): Likewise. * ast/rust-expr.h (class BreakExpr): Make expr in break optional. (class ReturnExpr): Make returned expr explicitely optional. Change the getters and introduce a const getter. * expand/rust-cfg-strip.cc (CfgStrip::visit): Convert to expected type. * expand/rust-macro-builtins-asm.cc (parse_reg_operand_in): Update constructor call to new expected types. (parse_reg_operand_out): Likewise. (parse_reg_operand_inout): Likewise. (parse_llvm_operands): Likewise. * expand/rust-macro-builtins-format-args.cc (format_args_parse_expr): Likewise. (format_args_parse_arguments): Likewise. * expand/rust-macro-builtins-helpers.cc (try_expand_many_expr): Update value for tl::expected. * expand/rust-macro-builtins-include.cc (MacroBuiltin::include_handler): Likewise. * expand/rust-macro-expand.cc (transcribe_expression): Update constructor call with expected value. * hir/rust-ast-lower-expr.cc (ASTLoweringExpr::visit): Likewise. * parse/rust-parse-error.h (enum class): Add new error types for Expr and StructExprField. * parse/rust-parse-impl-expr.hxx: Explicitely handle return expr parsing failure. Update to fit new expected types. * parse/rust-parse-impl.hxx: Likewise. * resolve/rust-late-name-resolver-2.0.cc (Late::visit): Adapt to tl::expected. * ast/rust-desugar-while-let.cc (DesugarWhileLet::DesugarCtx::make_break_arm): Use optional for break expressions when missing instead of nullptr. * parse/rust-parse.h: Change function return type with expected. Remove error state from ExprOrStmt. Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
GCC5 does not infer correctly the type to use within the template. gcc/rust/ChangeLog: * parse/rust-parse-impl-expr.hxx: Use explicit template. * parse/rust-parse-impl-path.hxx: Likewise. * parse/rust-parse-impl-ttree.hxx: Likewise. * parse/rust-parse-impl.hxx: Likewise. Signed-off-by: Pierre-Emmanuel Patry <[email protected]>
gcc/rust/ChangeLog: * resolve/rust-forever-stack.h: Add depth parameter to function. * resolve/rust-forever-stack.hxx: Likewise and use it.
…a prelude. gcc/rust/ChangeLog: * resolve/rust-early-name-resolver-2.0.cc (Early::finalize_glob_import): Save prelude if we find one. * resolve/rust-name-resolution-context.h: Add field. * resolve/rust-toplevel-name-resolver-2.0.cc (has_prelude_import): New function. (TopLevel::visit): Create a prelude glob import if necessary. * resolve/rust-toplevel-name-resolver-2.0.h: Allow glob imports to be prelude imports.
gcc/rust/ChangeLog: * resolve/rust-forever-stack.h: Add new resolve_path function. * resolve/rust-forever-stack.hxx: Implement it.
gcc/rust/ChangeLog: * resolve/rust-forever-stack.h: New function. * resolve/rust-forever-stack.hxx: Implement it. * resolve/rust-late-name-resolver-2.0.cc (Late::visit): Call it if the prelude exists and we have an unresolved Identifier Call it if the prelude exists and we have an unresolved Identifier.
We only want to emit the ones from regular name resolution as otherwise they will be doubled for the user for no good reason. gcc/rust/ChangeLog: * resolve/rust-name-resolution-context.h: Co-authored-by: Owen Avery <[email protected]>
Emit a diagnostic when #[link_name] is used without arguments, matching rustc behavior. This prevents silent acceptance of empty attributes and provides a helpful diagnostic that shows the expected form. Fixes #4228 gcc/rust/ChangeLog: * util/rust-attributes.cc: Emit diagnostic. gcc/testsuite/ChangeLog: * rust/compile/link_name-malformed.rs: New test. Signed-off-by: Jayant Chauhan <[email protected]>
gcc/rust/ChangeLog: * checks/lints/unused/rust-unused-checker.cc (UnusedChecker::visit): Emit warning in empty statement visitor. * checks/lints/unused/rust-unused-checker.h: Likewise. gcc/testsuite/ChangeLog: * rust/compile/redundant-semicolons_0.rs: New test. Signed-off-by: Lucas Ly Ba <[email protected]>
Emit a diagnostic when #[target_feature] is used without arguments, matching rustc behavior. This prevents silent acceptance of empty attributes and provides a helpful diagnostic that shows the expected form. Fixes #4233 gcc/rust/ChangeLog: * util/rust-attributes.cc (AttributeChecker::visit): Emit diagnostic. gcc/testsuite/ChangeLog: * rust/compile/target_feature-malformed-4233.rs: New test. Signed-off-by: Jayant Chauhan <[email protected]>
Emit a diagnostic when #[no_mangle] is used with arguments, matching rustc behavior. The no_mangle attribute is a word attribute and should not accept any input values. Fixes #4230 gcc/rust/ChangeLog: * util/rust-attributes.cc (AttributeChecker::visit): Emit diagnostic. gcc/testsuite/ChangeLog: * rust/compile/no_mangle-malformed.rs: New test. Signed-off-by: Jayant Chauhan <[email protected]>
gcc/rust/ChangeLog: * checks/lints/unused/rust-unused-checker.cc (UnusedChecker::visit_loop_label): Add warning for unused label in LoopLabel expr. * checks/lints/unused/rust-unused-checker.h: Likewise. * checks/lints/unused/rust-unused-collector.cc (UnusedCollector::visit): Check in BreakExpr and ContinueExpr if a label is used. * checks/lints/unused/rust-unused-collector.h: Likewise. * checks/lints/unused/rust-unused-context.cc (UnusedContext::add_label): Method helper. (UnusedContext::is_label_used): Likewise * checks/lints/unused/rust-unused-context.h: Likewise. gcc/testsuite/ChangeLog: * rust/compile/unused-label_0.rs: New test. Signed-off-by: Lucas Ly Ba <[email protected]>
Fix parsing of outer attributes in expressions. Fixes #3904 gcc/rust/ChangeLog: * parse/rust-parse-impl-expr.hxx(Parser::null_denotation): Add HASH case to handle outer attributes in expressions. Signed-off-by: lenny.chiadmi-delage <[email protected]>
Adds test of issue 3904 and fix test of the issue 3874 to follows rust 1.49 behavior. Fixes GCC-Rust/gccrs#3904 gcc/testsuite/ChangeLog: * rust/compile/issue-3874.rs: Fixes test. * rust/compile/issue-3904.rs: New test. Signed-off-by: lenny.chiadmi-delage <[email protected]>
gcc/rust/ChangeLog: * typecheck/rust-hir-type-check-expr.cc (visit(StructExprStruct)): Update to properly unwrap enum variants for type checking. * typecheck/rust-tyty.cc (VariantDef::get_fields) : Remove NUM assert. * backend/rust-compile-expr.cc: Update to properly unwrap enum variants for type resolution checking. Signed-off-by: Yap Zhi Heng <[email protected]>
11b099a to
2688e65
Compare
Member
Author
Same as current master branch (a1ee38a) |
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.
This is a fake PR, not meant to be merged. It tries to merge commits to upstream with an upstream base branch dkm/upstream-base.
We're only interested by the CI results.
-- gerris 🦀