]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
gccrs: Change crate_num_to_nodeid return type
authorPierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
Thu, 25 Apr 2024 14:15:42 +0000 (16:15 +0200)
committerArthur Cohen <arthur.cohen@embecosm.com>
Mon, 17 Mar 2025 15:35:24 +0000 (16:35 +0100)
Change the function's return type to an optional.

gcc/rust/ChangeLog:

* resolve/rust-ast-resolve-toplevel.h: Adapt the code to the new
return type.
* rust-session-manager.cc (Session::load_extern_crate): Likewise.
* util/rust-hir-map.cc (Mappings::crate_num_to_nodeid): Change the
return type.
* util/rust-hir-map.h: Update the function's prototype.

Signed-off-by: Pierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
gcc/rust/resolve/rust-ast-resolve-toplevel.h
gcc/rust/rust-session-manager.cc
gcc/rust/util/rust-hir-map.cc
gcc/rust/util/rust-hir-map.h

index 99fc4f6bfcc81d4fa80ce06a92d6129205e67e09..565ca922e84d91f031aeb162bdce1b80d6ff7418 100644 (file)
@@ -429,8 +429,7 @@ public:
     if (extern_crate.references_self ())
       {
        CrateNum crate_num = mappings.get_current_crate ();
-       bool ok = mappings.crate_num_to_nodeid (crate_num, resolved_crate);
-       rust_assert (ok);
+       resolved_crate = mappings.crate_num_to_nodeid (crate_num).value ();
       }
     else
       {
@@ -442,11 +441,9 @@ public:
                           extern_crate.get_referenced_crate ().c_str ());
            return;
          }
-       auto found_crate_num = cnum.value ();
-
-       bool ok
-         = mappings.crate_num_to_nodeid (found_crate_num, resolved_crate);
-       if (!ok)
+       if (auto resolved = mappings.crate_num_to_nodeid (*cnum))
+         resolved_crate = resolved.value ();
+       else
          {
            rust_internal_error_at (extern_crate.get_locus (),
                                    "failed to resolve crate to nodeid");
index da66479d608cc31a8700d00d56b73ac470b3add8..61a76d69a6b80d5f5518fb539f8cbeb82c1a10b1 100644 (file)
@@ -1050,12 +1050,10 @@ Session::load_extern_crate (const std::string &crate_name, location_t locus)
   // has it already been loaded?
   if (auto crate_num = mappings.lookup_crate_name (crate_name))
     {
-      NodeId resolved_node_id = UNKNOWN_NODEID;
-      bool resolved
-       = mappings.crate_num_to_nodeid (*crate_num, resolved_node_id);
-      rust_assert (resolved);
+      auto resolved_node_id = mappings.crate_num_to_nodeid (*crate_num);
+      rust_assert (resolved_node_id);
 
-      return resolved_node_id;
+      return *resolved_node_id;
     }
 
   std::string relative_import_path = "";
index 95d3b3a4d611d2994fc95132c16b8c2c7c6ece63..76642a1b13206b08fd22986e75b564ad7b0bd55d 100644 (file)
@@ -175,15 +175,14 @@ Mappings::lookup_crate_name (const std::string &crate_name) const
   return tl::nullopt;
 }
 
-bool
-Mappings::crate_num_to_nodeid (const CrateNum &crate_num, NodeId &node_id) const
+tl::optional<NodeId>
+Mappings::crate_num_to_nodeid (const CrateNum &crate_num) const
 {
   auto it = ast_crate_mappings.find (crate_num);
   if (it == ast_crate_mappings.end ())
-    return false;
+    return tl::nullopt;
 
-  node_id = it->second->get_node_id ();
-  return true;
+  return it->second->get_node_id ();
 }
 
 bool
index a68d81f34efe5ccf9aa920d4dca792a15d0ef9dd..8181abe00a9848a95d07ae69d97f3ff392e609ac 100644 (file)
@@ -85,7 +85,7 @@ public:
   std::string get_current_crate_name () const;
   tl::optional<CrateNum>
   lookup_crate_name (const std::string &crate_name) const;
-  bool crate_num_to_nodeid (const CrateNum &crate_num, NodeId &node_id) const;
+  tl::optional<NodeId> crate_num_to_nodeid (const CrateNum &crate_num) const;
   bool node_is_crate (NodeId node_id) const;
 
   NodeId get_next_node_id ();