Fix shareability of internalized nulls

src/wasm/literal.cpp

@@ -2700,11 +2700,12 @@ Literal Literal::externalize() const {
 }
 
 Literal Literal::internalize() const {
- auto extType = HeapTypes::ext.getBasic(type.getHeapType().getShared());
+ auto share = type.getHeapType().getShared();
+ auto extType = HeapTypes::ext.getBasic(share);
  assert(Type::isSubType(type, Type(extType, Nullable)) &&
  "can only internalize external references");
  if (isNull()) {
- return Literal(std::shared_ptr<GCData>{}, HeapType::none);
+ return Literal(std::shared_ptr<GCData>{}, HeapTypes::none.getBasic(share));
  }
  if (gcData->type.isMaybeShared(HeapType::i31)) {
  assert(gcData->values[0].type.getHeapType().isMaybeShared(HeapType::i31));

test/spec/convert_extern.wast

@@ -0,0 +1,11 @@
+(module
+ (func $shared-null (export "shared-null") (result (ref null (shared any)))
+ ;; The shared null here should remain shared as we internalize it.
+ (any.convert_extern
+ (ref.null (shared noextern))
+ )
+ )
+)
+
+(assert_return (invoke "shared-null") (ref.null (shared any)))
+