Commit d6f859b3 authored by Andreas Haas's avatar Andreas Haas Committed by Commit Bot

Remove resolved TODO

As far as I understand the TODO, it has been resolved already some lines
below:

if (kEnableDebug) {
  VerifyCodeRangeIsDisjoint(data);
}

bug: v8:8015
R=titzer@chromium.org

Change-Id: I3686ad609b7c04e56b14ad2d1ccb265ac260bac7
Reviewed-on: https://chromium-review.googlesource.com/1185012Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
Commit-Queue: Andreas Haas <ahaas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55311}
parent 258ec2c5
...@@ -134,7 +134,6 @@ CodeProtectionInfo* CreateHandlerData( ...@@ -134,7 +134,6 @@ CodeProtectionInfo* CreateHandlerData(
int RegisterHandlerData( int RegisterHandlerData(
Address base, size_t size, size_t num_protected_instructions, Address base, size_t size, size_t num_protected_instructions,
const ProtectedInstructionData* protected_instructions) { const ProtectedInstructionData* protected_instructions) {
// TODO(eholk): in debug builds, make sure this data isn't already registered.
CodeProtectionInfo* data = CreateHandlerData( CodeProtectionInfo* data = CreateHandlerData(
base, size, num_protected_instructions, protected_instructions); base, size, num_protected_instructions, protected_instructions);
......
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