diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2023-02-13 11:15:24 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-02-13 11:15:24 +0000 |
commit | 59d4ee6cc1b14ddc01d062a41bec364e959793c7 (patch) | |
tree | dc3137f8905d5ad3cebefb6b766859d49944a9eb /gcc/rust/ast/rust-ast.cc | |
parent | 3a8c8d0d41b57fbb49ab39715b70495d5d1e8dd1 (diff) | |
parent | f67f5409d2fe4191ab24b5eb634c26306386fb25 (diff) | |
download | gcc-59d4ee6cc1b14ddc01d062a41bec364e959793c7.zip gcc-59d4ee6cc1b14ddc01d062a41bec364e959793c7.tar.gz gcc-59d4ee6cc1b14ddc01d062a41bec364e959793c7.tar.bz2 |
Merge #1750
1750: Implement fixed point macro expansion r=CohenArthur a=CohenArthur
This is a cleanup of #1606's branch, which also contains the code necessary for performing eager macro expansion in builtin macros.
This commit changes our macro expansion system from an eager and recursive
macro expansion to a fixed-point like system. Instead of, when seeing
a macro invocation, expanding it and all of the macros within it, we
now perform multiple passes of expansion on the entire crate.
This, however, leads to a problem. Rust macros are expanded lazily, but
Rust builtin macros should be expanded eagerly. Due to this, we must
work around the lazy expansion in builtin macros and perform eager
expansion for each pass of the fixed-point, before finally expanding
the builtin when there are no longer any inner macro invocations.
To perform proper macro scoping, the ENR now keeps track of the current
scope (`current_scope` member) and resolves macros accordingly.
This is done through the use of the `scoped` method, which creates a new
scope, runs a specified lambda and then exits the scope. This prevents
pushing/popping errors that we've seen happen already in similar
contexts.
We might think about generalizing it to other classes, providing a
`Scoped<EntryFn, ExitFn>` class or similar
Fixes #1795
Co-authored-by: Arthur Cohen <arthur.cohen@embecosm.com>
Diffstat (limited to 'gcc/rust/ast/rust-ast.cc')
-rw-r--r-- | gcc/rust/ast/rust-ast.cc | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/gcc/rust/ast/rust-ast.cc b/gcc/rust/ast/rust-ast.cc index 813c35b..a5b7f60b 100644 --- a/gcc/rust/ast/rust-ast.cc +++ b/gcc/rust/ast/rust-ast.cc @@ -1323,6 +1323,12 @@ std::string MacroInvocation::as_string () const { std::string str = "MacroInvocation: "; + auto is_builtin = kind == InvocKind::Builtin; + + if (is_builtin) + str += "[builtin] "; + else + str += "[regular] "; str += append_attributes (outer_attrs, OUTER); @@ -1331,6 +1337,16 @@ MacroInvocation::as_string () const str += "\n has semicolon: "; str += has_semicolon () ? "true" : "false"; + if (is_builtin) + { + str += "[PENDING EAGER INVOCATIONS]: "; + for (auto &pending : pending_eager_invocs) + { + str += pending->as_string (); + str += "\n"; + } + } + return str; } |