[wasm] Trap handling: ProtectedLoad instruction

src/compiler/machine-operator.h

Index: src/compiler/machine-operator.h
diff --git a/src/compiler/machine-operator.h b/src/compiler/machine-operator.h
index 5d0e677f8d5ec99b325c1799b418eb5b05e6f62d..b7748bfffb892311212cc96c6489a4181c8c662a 100644
--- a/src/compiler/machine-operator.h
+++ b/src/compiler/machine-operator.h
@@ -41,6 +41,7 @@ class OptionalOperator final {
 
 // A Load needs a MachineType.
 typedef MachineType LoadRepresentation;
+typedef LoadRepresentation ProtectedLoadRepresentation;

[Mircea Trofin, 2016/09/02 04:05:45]

What's the value of the typedef - just shorter? If so, I'd prefer the long name,
it improves readability.

[Eric Holk, 2016/09/02 20:05:31]

This is actually defining ProtectedLoadRepresentation as an alias of
LoadRepresentation. Just using LoadRepresentation is probably fine, but I did it
this way for consistency with how The other loads like CheckedLoad work. See
machine-operator.cc around line 505.

 
 LoadRepresentation LoadRepresentationOf(Operator const*);
 
@@ -601,6 +602,7 @@ class MachineOperatorBuilder final : public ZoneObject {
 
   // load [base + index]
   const Operator* Load(LoadRepresentation rep);
+  const Operator* ProtectedLoad(LoadRepresentation rep);
 
   // store [base + index], value
   const Operator* Store(StoreRepresentation rep);