Skip to content

Commit

Permalink
Bug 1720781 - Part 1: Remove no longer needed CheckObjCoercible. r=iain
Browse files Browse the repository at this point in the history
This check is no longer needed after <tc39/ecma262#2267>.

Differential Revision: https://phabricator.services.mozilla.com/D120609
  • Loading branch information
anba committed Jul 23, 2021
1 parent a4a5bf6 commit 7475a0a
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions js/src/frontend/ElemOpEmitter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,6 @@ bool ElemOpEmitter::prepareForObj() {
bool ElemOpEmitter::prepareForKey() {
MOZ_ASSERT(state_ == State::Obj);

if (!isSuper() && isIncDec()) {
if (!bce_->emit1(JSOp::CheckObjCoercible)) {
// [stack] OBJ
return false;
}
}
if (isCall()) {
if (!bce_->emit1(JSOp::Dup)) {
// [stack] # if Super
Expand Down

0 comments on commit 7475a0a

Please sign in to comment.