Commit 0b344e47 authored by Marja Hölttä's avatar Marja Hölttä Committed by Commit Bot

[cleanup] Remove 2 x TODO(marja)

Do one, the other one was obsolete.

Bug: v8:11074
Change-Id: I6f42aade9d6413f754ff5821ae9394045166eaa2
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2521151Reviewed-by: 's avatarRoss McIlroy <rmcilroy@chromium.org>
Commit-Queue: Marja Hölttä <marja@chromium.org>
Cr-Commit-Position: refs/heads/master@{#71004}
parent c35d70dd
......@@ -9,10 +9,6 @@ namespace error {
transitioning javascript builtin AggregateErrorConstructor(
js-implicit context: NativeContext, target: JSFunction,
newTarget: JSAny)(...arguments): JSAny {
// This function is implementing the spec as suggested by
// https://github.com/tc39/proposal-promise-any/pull/59 . FIXME(marja):
// change this if the PR is declined, otherwise remove the comment.
// 1. If NewTarget is undefined, let newTarget be the active function
// object, else let newTarget be NewTarget.
// 2. Let O be ? OrdinaryCreateFromConstructor(newTarget,
......
......@@ -77,8 +77,7 @@ Register BytecodeArrayBuilder::Receiver() const {
}
Register BytecodeArrayBuilder::Local(int index) const {
// TODO(marja): Make a DCHECK once crbug.com/706234 is fixed.
CHECK_LT(index, locals_count());
DCHECK_LT(index, locals_count());
return Register(index);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment