Reland "[build][torque] remove workarounds for clang bug"

This is a reland of c5154eea

Now, the CL no longer enables ASAN for Torque, since this seems
to be another Clang issue that's not fixed yet.

Original change's description:
> [build][torque] remove workarounds for clang bug
>
> Now that https://bugs.llvm.org/show_bug.cgi?id=40118 has been fixed and
> rolled into V8, we can remove the workarounds for this Clang bug.
>
> This also effectively reverts
> https://chromium-review.googlesource.com/c/v8/v8/+/1280222
>
> Bug: chromium:893437
> Change-Id: Ia0d6d8ebdafafbc380b1b7a7809ef16effe50d71
> Reviewed-on: https://chromium-review.googlesource.com/c/1425519
> Reviewed-by: Jaroslav Sevcik <jarin@chromium.org>
> Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#58987}

Bug: chromium:893437
TBR: jarin@chromium.org
Change-Id: Ib9ac101702d12e5bf28891cbe6b5b16bd9d5e402
Reviewed-on: https://chromium-review.googlesource.com/c/1433787
Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#59090}

src/torque/instructions.h

@@ -207,10 +207,7 @@ struct LoadObjectFieldInstruction : InstructionBase {
   TORQUE_INSTRUCTION_BOILERPLATE()
   LoadObjectFieldInstruction(const ClassType* class_type,
                              std::string field_name)
-      : class_type(class_type) {
-    // The normal way to write this triggers a bug in Clang on Windows.
-    this->field_name = std::move(field_name);
-  }
+      : class_type(class_type), field_name(std::move(field_name)) {}
   const ClassType* class_type;
   std::string field_name;
 };
@@ -219,10 +216,7 @@ struct StoreObjectFieldInstruction : InstructionBase {
   TORQUE_INSTRUCTION_BOILERPLATE()
   StoreObjectFieldInstruction(const ClassType* class_type,
                               std::string field_name)
-      : class_type(class_type) {
-    // The normal way to write this triggers a bug in Clang on Windows.
-    this->field_name = std::move(field_name);
-  }
+      : class_type(class_type), field_name(std::move(field_name)) {}
   const ClassType* class_type;
   std::string field_name;
 };
@@ -399,10 +393,8 @@ struct ReturnInstruction : InstructionBase {
 
 struct PrintConstantStringInstruction : InstructionBase {
   TORQUE_INSTRUCTION_BOILERPLATE()
-  explicit PrintConstantStringInstruction(std::string message) {
-    // The normal way to write this triggers a bug in Clang on Windows.
-    this->message = std::move(message);
-  }
+  explicit PrintConstantStringInstruction(std::string message)
+      : message(std::move(message)) {}
 
   std::string message;
 };
@@ -411,10 +403,8 @@ struct AbortInstruction : InstructionBase {
   TORQUE_INSTRUCTION_BOILERPLATE()
   enum class Kind { kDebugBreak, kUnreachable, kAssertionFailure };
   bool IsBlockTerminator() const override { return kind != Kind::kDebugBreak; }
-  explicit AbortInstruction(Kind kind, std::string message = "") : kind(kind) {
-    // The normal way to write this triggers a bug in Clang on Windows.
-    this->message = std::move(message);
-  }
+  explicit AbortInstruction(Kind kind, std::string message = "")
+      : kind(kind), message(std::move(message)) {}
 
   Kind kind;
   std::string message;