Commit 6adb9f29 authored by Georg Schmid's avatar Georg Schmid Committed by Commit Bot

Remove undefined value workaround in AllocateFastLiteral

R=bmeurer@chromium.org

Bug: v8:9299
Change-Id: I77e8b201bd5fd1c544cced301c8805571c4c1167
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1632212Reviewed-by: 's avatarBenedikt Meurer <bmeurer@chromium.org>
Commit-Queue: Georg Schmid <gsps@google.com>
Cr-Commit-Position: refs/heads/master@{#61890}
parent 0b14b8a1
......@@ -1630,11 +1630,7 @@ Node* JSCreateLowering::AllocateFastLiteral(Node* effect, Node* control,
access.machine_type = MachineType::Float64();
access.type = Type::Number();
uint64_t value_bits = boilerplate.RawFastDoublePropertyAsBitsAt(index);
// TODO(gsps): Remove the is_undefined once we've eliminated the case
// where a pointer to the undefined value gets reinterpreted as the
// boilerplate value of certain uninitialized unboxed double fields.
bool is_undefined = value_bits == (*(factory()->undefined_value())).ptr();
if (value_bits == kHoleNanInt64 || is_undefined) {
if (value_bits == kHoleNanInt64) {
// This special case is analogous to is_uninitialized being true in the
// non-unboxed-double case below. The store of the hole NaN value here
// will always be followed by another store that actually initializes
......
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