[zone] Cleanup zone allocations in src/asmjs

... by migrating old-style code
  MyObject* obj = new (zone) MyObject(...)

to the new style
  MyObject* obj = zone->New<MyObject>(...)

Bug: v8:10689
Change-Id: I8293cc1854e39726eb9e4e5f9d2a34d38fa54859
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2288861Reviewed-by: Clemens Backes <clemensb@chromium.org>
Commit-Queue: Igor Sheludko <ishell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#68791}

src/asmjs/asm-js.cc

@@ -241,9 +241,9 @@ UnoptimizedCompilationJob::Status AsmJsCompilationJob::ExecuteJobImpl() {
     }
     return FAILED;
   }
-  module_ = new (compile_zone) wasm::ZoneBuffer(compile_zone);
+  module_ = compile_zone->New<wasm::ZoneBuffer>(compile_zone);
   parser.module_builder()->WriteTo(module_);
-  asm_offsets_ = new (compile_zone) wasm::ZoneBuffer(compile_zone);
+  asm_offsets_ = compile_zone->New<wasm::ZoneBuffer>(compile_zone);
   parser.module_builder()->WriteAsmJsOffsetTable(asm_offsets_);
   stdlib_uses_ = *parser.stdlib_uses();
 

src/asmjs/asm-parser.cc

@@ -74,7 +74,7 @@ AsmJsParser::AsmJsParser(Zone* zone, uintptr_t stack_limit,
                          Utf16CharacterStream* stream)
     : zone_(zone),
       scanner_(stream),
-      module_builder_(new (zone) WasmModuleBuilder(zone)),
+      module_builder_(zone->New<WasmModuleBuilder>(zone)),
       return_type_(nullptr),
       stack_limit_(stack_limit),
       global_var_info_(zone),
@@ -564,8 +564,7 @@ void AsmJsParser::ValidateModuleVarImport(VarInfo* info,
       AddGlobalImport(name, AsmType::Int(), kWasmI32, mutable_variable, info);
     } else {
       info->kind = VarKind::kImportedFunction;
-      info->import = new (zone()->New(sizeof(FunctionImportInfo)))
-          FunctionImportInfo(name, zone());
+      info->import = zone()->New<FunctionImportInfo>(name, zone());
       info->mutable_variable = false;
     }
   }

src/asmjs/asm-types.cc

@@ -172,7 +172,7 @@ class AsmFroundType final : public AsmCallableType {
 }  // namespace
 
 AsmType* AsmType::FroundType(Zone* zone) {
-  auto* Fround = new (zone) AsmFroundType();
+  auto* Fround = zone->New<AsmFroundType>();
   return reinterpret_cast<AsmType*>(Fround);
 }
 
@@ -195,6 +195,7 @@ namespace {
 class AsmMinMaxType final : public AsmCallableType {
  private:
   friend AsmType;
+  friend Zone;
 
   AsmMinMaxType(AsmType* dest, AsmType* src)
       : AsmCallableType(), return_type_(dest), arg_(src) {}
@@ -231,7 +232,7 @@ class AsmMinMaxType final : public AsmCallableType {
 AsmType* AsmType::MinMaxType(Zone* zone, AsmType* dest, AsmType* src) {
   DCHECK_NOT_NULL(dest->AsValueType());
   DCHECK_NOT_NULL(src->AsValueType());
-  auto* MinMax = new (zone) AsmMinMaxType(dest, src);
+  auto* MinMax = zone->New<AsmMinMaxType>(dest, src);
   return reinterpret_cast<AsmType*>(MinMax);
 }
 

src/asmjs/asm-types.h

@@ -139,6 +139,7 @@ class V8_EXPORT_PRIVATE AsmFunctionType final : public AsmCallableType {
 
  private:
   friend AsmType;
+  friend Zone;
 
   std::string Name() override;
   bool IsA(AsmType* other) override;
@@ -160,6 +161,7 @@ class V8_EXPORT_PRIVATE AsmOverloadedFunctionType final
 
  private:
   friend AsmType;
+  friend Zone;
 
   explicit AsmOverloadedFunctionType(Zone* zone) : overloads_(zone) {}
 
@@ -196,14 +198,14 @@ class V8_EXPORT_PRIVATE AsmType {
   // A function returning ret. Callers still need to invoke AddArgument with the
   // returned type to fully create this type.
   static AsmType* Function(Zone* zone, AsmType* ret) {
-    AsmFunctionType* f = new (zone) AsmFunctionType(zone, ret);
+    AsmFunctionType* f = zone->New<AsmFunctionType>(zone, ret);
     return reinterpret_cast<AsmType*>(f);
   }
 
   // Overloaded function types. Not creatable by asm source, but useful to
   // represent the overloaded stdlib functions.
   static AsmType* OverloadedFunction(Zone* zone) {
-    auto* f = new (zone) AsmOverloadedFunctionType(zone);
+    auto* f = zone->New<AsmOverloadedFunctionType>(zone);
     return reinterpret_cast<AsmType*>(f);
   }