test/unittests/compiler/change-lowering-unittest.cc - Issue 795713003: Steps towards unification of number bitset and range types.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: test/unittests/compiler/change-lowering-unittest.cc

Issue 795713003: Steps towards unification of number bitset and range types. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master

Patch Set: Fix BitsetType::Max for OtherNumber with missing representation Created 6 years ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

Index: test/unittests/compiler/change-lowering-unittest.cc

diff --git a/test/unittests/compiler/change-lowering-unittest.cc b/test/unittests/compiler/change-lowering-unittest.cc

index c6b069536448b152fdd20f0d8e3688c650465110..8cac43dece278ac97b7d7c136c4f9506f9c47601 100644

--- a/test/unittests/compiler/change-lowering-unittest.cc

+++ b/test/unittests/compiler/change-lowering-unittest.cc

@@ -216,7 +216,7 @@ TARGET_TEST_F(ChangeLowering32Test, ChangeInt32ToTagged) {

TARGET_TEST_F(ChangeLowering32Test, ChangeInt32ToTaggedSmall) {

Node* val = Parameter(0);

Node* node = graph()->NewNode(simplified()->ChangeInt32ToTagged(), val);

- NodeProperties::SetBounds(val, Bounds(Type::None(), Type::SignedSmall()));

+ NodeProperties::SetBounds(val, Bounds(Type::None(), Type::Signed31()));

Reduction reduction = Reduce(node);

ASSERT_TRUE(reduction.Changed());

« test/cctest/test-types.cc ('K') | « test/cctest/types-fuzz.h ('k') | test/unittests/compiler/js-builtin-reducer-unittest.cc » ('j') | no next file with comments »