[MERGE #5198 @MikeHolman] fix bad merge that removed branch from lower

MikeHolman · MikeHolman · commit 22e34d8a37ad · 2018-05-21T10:29:19.000-07:00
Merge pull request #5198 from MikeHolman:fixmerge Security fix was supposed to move the branch, but it got removed during merge
diff --git a/lib/Backend/Lower.cpp b/lib/Backend/Lower.cpp
@@ -15733,6 +15733,8 @@ Lowerer::GeneratePropertyStringTest(IR::RegOpnd *srcReg, IR::Instr *instrInsert,
 
     InsertObjectPoison(srcReg, branchInstr, instrInsert, !usePoison);
 
+    InsertBranch(Js::OpCode::Br, propStrLoadedLabel, instrInsert);
+
     instrInsert->InsertBefore(notPropStrLabel);
 
     branchInstr = InsertCompareBranch(
