Fix Switch-related bug in instrSuccOffsets

Summary: The offsets are relative to the Switch instruction. Reviewed By: @jdelong Differential Revision: D1848966
aosvaldtRS · Feb 18, 2015 · 9f0256e · 9f0256e
1 parent bd4e0be
commit 9f0256e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/hphp/runtime/vm/hhbc.cpp b/hphp/runtime/vm/hhbc.cpp
@@ -402,7 +402,7 @@ OffsetSet instrSuccOffsets(Op* opc, const Unit* unit) {
 
   if (isSwitch(*opc)) {
     foreachSwitchTarget(opc, [&](Offset& offset) {
-        succBcOffs.insert(offset);
+        succBcOffs.insert(offset + opc - bcStart);
       });
   } else {
     Offset target = instrJumpTarget(bcStart, opc - bcStart);