Skip to content

Add assert macro handler#4513

Open
P-E-P wants to merge 1 commit intoRust-GCC:masterfrom
P-E-P:fix_parser_bug_null_denotation_macro
Open

Add assert macro handler#4513
P-E-P wants to merge 1 commit intoRust-GCC:masterfrom
P-E-P:fix_parser_bug_null_denotation_macro

Conversation

@P-E-P
Copy link
Copy Markdown
Member

@P-E-P P-E-P commented Apr 3, 2026

Assert macro handler was missing, insert a basic handler that desugars to a condition and a call to panic.

@P-E-P P-E-P marked this pull request as draft April 3, 2026 14:59
@P-E-P P-E-P force-pushed the fix_parser_bug_null_denotation_macro branch 2 times, most recently from aec63b4 to 5a3a686 Compare April 13, 2026 14:38
@P-E-P P-E-P marked this pull request as ready for review April 13, 2026 14:38
@P-E-P P-E-P force-pushed the fix_parser_bug_null_denotation_macro branch from 5a3a686 to 11e52fb Compare April 13, 2026 15:05
@P-E-P P-E-P requested a review from CohenArthur April 13, 2026 16:18
@P-E-P
Add assert macro handler
436fe22 
Assert macro handler was missing, insert a basic handler that desugars to
a condition and a call to panic.

gcc/rust/ChangeLog:

	* expand/rust-macro-builtins-log-debug.cc (MacroBuiltin::assert_handler):
	Add basic assert builtin macro handler.

gcc/testsuite/ChangeLog:

	* rust/compile/assert_missing_panic.rs: New test.

Signed-off-by: Pierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
@P-E-P P-E-P force-pushed the fix_parser_bug_null_denotation_macro branch from 11e52fb to 436fe22 Compare April 13, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CohenArthur CohenArthur Awaiting requested review from CohenArthur

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@P-E-P