[NFC] Take a HeapType instead of Type in RefFunc::finalize (#7442)

This removes from the callers the burden of constructing the type with
the correct nullability and other future attributes.

Author: tlively

src/passes/Precompute.cpp

@@ -313,7 +313,7 @@ struct Precompute
           if (auto* r = curr->value->template dynCast<RefFunc>()) {
             r->func = singleValue.getFunc();
             auto heapType = getModule()->getFunction(r->func)->type;
-            r->finalize(Type(heapType, NonNullable));
+            r->finalize(heapType);
             curr->finalize();
             return;
           }

src/wasm-builder.h

@@ -688,7 +688,7 @@ class Builder {
   RefFunc* makeRefFunc(Name func, HeapType heapType) {
     auto* ret = wasm.allocator.alloc<RefFunc>();
     ret->func = func;
-    ret->finalize(Type(heapType, NonNullable));
+    ret->finalize(heapType);
     return ret;
   }
   RefEq* makeRefEq(Expression* left, Expression* right) {

src/wasm.h

@@ -1364,7 +1364,7 @@ class RefFunc : public SpecificExpression<Expression::RefFuncId> {
   Name func;
 
   void finalize();
-  void finalize(Type type_);
+  void finalize(HeapType heapType);
 };
 
 class RefEq : public SpecificExpression<Expression::RefEqId> {

src/wasm/wasm.cpp

@@ -821,7 +821,9 @@ void RefFunc::finalize() {
   assert(type.isSignature());
 }
 
-void RefFunc::finalize(Type type_) { type = type_; }
+void RefFunc::finalize(HeapType heapType) {
+  type = Type(heapType, NonNullable);
+}
 
 void RefEq::finalize() {
   if (left->type == Type::unreachable || right->type == Type::unreachable) {