diff options
| author | Sourabh Jaiswal <sourabhrj31@gmail.com> | 2024-03-01 19:46:23 +0800 |
|---|---|---|
| committer | CohenArthur <arthur.cohen@embecosm.com> | 2024-03-05 10:26:25 +0000 |
| commit | 452345f0f88dddbb0ab9b2f31b6239af6288a984 (patch) | |
| tree | 5d0951aee752770746031b953c33fb60b71cbedd /gcc/rust/expand | |
| parent | dae69e86c0feafd4e67bec2309c8e14b82f935ac (diff) | |
| download | gcc-452345f0f88dddbb0ab9b2f31b6239af6288a984.zip gcc-452345f0f88dddbb0ab9b2f31b6239af6288a984.tar.gz gcc-452345f0f88dddbb0ab9b2f31b6239af6288a984.tar.bz2 | |
Clean BiMap to use tl::optional for lookups
gcc/rust/Changelog:
* expand/rust-expand-visitor.cc
(ExpandVisitor::expand_inner_items): Adjust to use has_value ()
(ExpandVisitor::expand_inner_stmts): Likewise
* expand/rust-macro-builtins.cc (builtin_macro_from_string): Likewise
(make_macro_path_str): Likewise
* util/rust-hir-map.cc (Mappings::insert_macro_def): Likewise
* util/rust-lang-item.cc (LangItem::Parse): Adjust to return tl::optional
(LangItem::toString) Likewise
* util/rust-token-converter.cc (handle_suffix): Adjust to use value.or ()
(from_literal) Likewise
* util/bi-map.h (BiMap::lookup): Adjust to use tl::optional for
lookups
Signed-off-by: Sourabh Jaiswal <sourabhrj31@gmail.com>
Diffstat (limited to 'gcc/rust/expand')
| -rw-r--r-- | gcc/rust/expand/rust-expand-visitor.cc | 8 | ||||
| -rw-r--r-- | gcc/rust/expand/rust-macro-builtins.cc | 8 |
2 files changed, 8 insertions, 8 deletions
diff --git a/gcc/rust/expand/rust-expand-visitor.cc b/gcc/rust/expand/rust-expand-visitor.cc index 6ca6311..bd49fd9 100644 --- a/gcc/rust/expand/rust-expand-visitor.cc +++ b/gcc/rust/expand/rust-expand-visitor.cc @@ -186,11 +186,11 @@ ExpandVisitor::expand_inner_items ( { auto maybe_builtin = MacroBuiltin::builtins.lookup ( to_derive.get ().as_string ()); - if (MacroBuiltin::builtins.is_iter_ok (maybe_builtin)) + if (maybe_builtin.has_value ()) { auto new_item = builtin_derive_item (*item, current, - maybe_builtin->second); + maybe_builtin.value ()); // this inserts the derive *before* the item - is it a // problem? it = items.insert (it, std::move (new_item)); @@ -272,11 +272,11 @@ ExpandVisitor::expand_inner_stmts (AST::BlockExpr &expr) { auto maybe_builtin = MacroBuiltin::builtins.lookup ( to_derive.get ().as_string ()); - if (MacroBuiltin::builtins.is_iter_ok (maybe_builtin)) + if (maybe_builtin.has_value ()) { auto new_item = builtin_derive_item (item, current, - maybe_builtin->second); + maybe_builtin.value ()); // this inserts the derive *before* the item - is it a // problem? it = stmts.insert (it, std::move (new_item)); diff --git a/gcc/rust/expand/rust-macro-builtins.cc b/gcc/rust/expand/rust-macro-builtins.cc index e4ca0d8..8cf3205 100644 --- a/gcc/rust/expand/rust-macro-builtins.cc +++ b/gcc/rust/expand/rust-macro-builtins.cc @@ -142,9 +142,9 @@ tl::optional<BuiltinMacro> builtin_macro_from_string (const std::string &identifier) { auto macro = MacroBuiltin::builtins.lookup (identifier); - rust_assert (MacroBuiltin::builtins.is_iter_ok (macro)); + rust_assert (macro.has_value ()); - return macro->second; + return macro; } namespace { @@ -152,9 +152,9 @@ std::string make_macro_path_str (BuiltinMacro kind) { auto str = MacroBuiltin::builtins.lookup (kind); - rust_assert (MacroBuiltin::builtins.is_iter_ok (str)); + rust_assert (str.has_value ()); - return str->second; + return str.value (); } static std::vector<std::unique_ptr<AST::MacroInvocation>> |