Skip to content

Commit

Permalink
[MERGE #6123 @MikeHolman] fix potentially bad merge from 1.11 to master
Browse files Browse the repository at this point in the history
Merge pull request #6123 from pr/MikeHolman/mergefix

There was extra condition for lazy bailout in master. When merging servicing update into unreleased/future, I noticed that the condition was handled different than how I merged it in master.
  • Loading branch information
MikeHolman committed May 15, 2019
2 parents 1c75087 + ea16256 commit 49f594b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/Backend/Lower.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4865,11 +4865,11 @@ bool Lowerer::TryLowerNewScObjectWithFixedCtorCache(IR::Instr* newObjInstr, IR::

JITTimeConstructorCache * ctorCache;

if (newObjInstr->HasBailOutInfo() && !newObjInstr->HasLazyBailOut() && newObjInstr->GetBailOutKindNoBits() == IR::BailOutFailedCtorGuardCheck)
if (newObjInstr->HasBailOutInfo() && newObjInstr->GetBailOutKindNoBits() == IR::BailOutFailedCtorGuardCheck)
{
Assert(newObjInstr->IsNewScObjectInstr());
Assert(newObjInstr->IsProfiledInstr());
Assert(newObjInstr->GetBailOutKindNoBits() == IR::BailOutFailedCtorGuardCheck || newObjInstr->HasLazyBailOut());
Assert(newObjInstr->GetBailOutKind() == IR::BailOutFailedCtorGuardCheck);

emitBailOut = true;

Expand Down

0 comments on commit 49f594b

Please sign in to comment.