enabling ext ops in asmjs intr

Krovatkin · Krovatkin · commit 299309d8be46 · 2016-05-02T11:49:30.000-07:00
adding a unittest

adding the copyright and dealing with whitespaces and dup newlines

fix formatting
diff --git a/lib/Runtime/Language/InterpreterLoop.inl b/lib/Runtime/Language/InterpreterLoop.inl
@@ -233,7 +233,6 @@ SWAP_BP_FOR_OPCODE:
             }
             case INTERPRETER_OPCODE::ExtendedMediumLayoutPrefix:
             {
-#ifndef INTERPRETER_ASMJS  // Asmjs doesn't have any extended opcodes for now, remove that case
                 ip = [this](const byte * ip) -> const byte *
                 {
                     INTERPRETER_OPCODE op = (INTERPRETER_OPCODE)(ReadByteOp<INTERPRETER_OPCODE>(ip
@@ -262,7 +261,6 @@ SWAP_BP_FOR_OPCODE:
                     // Aborting the current interpreter loop to switch the profile mode
                     return nullptr;
                 }
-#endif
 #endif
                 break;
             }
@@ -309,7 +307,6 @@ SWAP_BP_FOR_OPCODE:
             }
             case INTERPRETER_OPCODE::ExtendedLargeLayoutPrefix:
             {
-#ifndef INTERPRETER_ASMJS  // Asmjs doesn't have any extended opcodes for now, remove that case
                 ip = [this](const byte * ip) -> const byte *
                 {
                     INTERPRETER_OPCODE op = (INTERPRETER_OPCODE)(ReadByteOp<INTERPRETER_OPCODE>(ip
@@ -338,7 +335,6 @@ SWAP_BP_FOR_OPCODE:
                     // Aborting the current interpreter loop to switch the profile mode
                     return nullptr;
                 }
-#endif
 #endif
                 break;
             }
diff --git a/test/Bugs/rlexe.xml b/test/Bugs/rlexe.xml
@@ -299,4 +299,12 @@
       <compile-flags> -bgjit- -simdjs -simd128typespec -asmjs- -mic:1 -lic:1 -off:simplejit</compile-flags>
     </default>
   </test>
+  <test>
+    <default>
+      <files>testSimdManyVars.js</files>
+      <baseline>testSimdManyVars.baseline</baseline>
+      <tags>exclude_dynapogo,exclude_ship</tags>
+      <compile-flags> -simdjs -asmjs -off:backend -testtrace:asmjs</compile-flags>
+    </default>
+  </test>
 </regress-exe>
diff --git a/test/Bugs/testSimdManyVars.baseline b/test/Bugs/testSimdManyVars.baseline
@@ -0,0 +1,2 @@
+Successfully compiled asm.js code
+PASS
diff --git a/test/Bugs/testSimdManyVars.js b/test/Bugs/testSimdManyVars.js

Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,6 @@ SWAP_BP_FOR_OPCODE:`
`233`	`233`	`}`
`234`	`234`	`case INTERPRETER_OPCODE::ExtendedMediumLayoutPrefix:`
`235`	`235`	`{`
`236`		`-#ifndef INTERPRETER_ASMJS // Asmjs doesn't have any extended opcodes for now, remove that case`
`237`	`236`	`ip = [this](const byte * ip) -> const byte *`
`238`	`237`	`{`
`239`	`238`	`INTERPRETER_OPCODE op = (INTERPRETER_OPCODE)(ReadByteOp<INTERPRETER_OPCODE>(ip`
`@@ -262,7 +261,6 @@ SWAP_BP_FOR_OPCODE:`
`262`	`261`	`// Aborting the current interpreter loop to switch the profile mode`
`263`	`262`	`return nullptr;`
`264`	`263`	`}`
`265`		`-#endif`
`266`	`264`	`#endif`
`267`	`265`	`break;`
`268`	`266`	`}`
`@@ -309,7 +307,6 @@ SWAP_BP_FOR_OPCODE:`
`309`	`307`	`}`
`310`	`308`	`case INTERPRETER_OPCODE::ExtendedLargeLayoutPrefix:`
`311`	`309`	`{`
`312`		`-#ifndef INTERPRETER_ASMJS // Asmjs doesn't have any extended opcodes for now, remove that case`
`313`	`310`	`ip = [this](const byte * ip) -> const byte *`
`314`	`311`	`{`
`315`	`312`	`INTERPRETER_OPCODE op = (INTERPRETER_OPCODE)(ReadByteOp<INTERPRETER_OPCODE>(ip`
`@@ -338,7 +335,6 @@ SWAP_BP_FOR_OPCODE:`
`338`	`335`	`// Aborting the current interpreter loop to switch the profile mode`
`339`	`336`	`return nullptr;`
`340`	`337`	`}`
`341`		`-#endif`
`342`	`338`	`#endif`
`343`	`339`	`break;`
`344`	`340`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+Successfully compiled asm.js code`
	`2`	`+PASS`