Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1059)

Unified Diff: runtime/vm/intermediate_language_test.cc

Issue 333643004: Add range analysis for left-shift smi operations. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: addressed comments, added tests Created 6 years, 6 months 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
« no previous file with comments | « runtime/vm/intermediate_language.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/intermediate_language_test.cc
===================================================================
--- runtime/vm/intermediate_language_test.cc (revision 37229)
+++ runtime/vm/intermediate_language_test.cc (working copy)
@@ -62,6 +62,58 @@
EXPECT(range_x->IsWithin(-15, 100));
EXPECT(!range_x->IsWithin(-15, 99));
EXPECT(!range_x->IsWithin(-14, 100));
+
+#define TEST_RANGE_OP(Op, l_min, l_max, r_min, r_max, result_min, result_max) \
+ { \
+ RangeBoundary min, max; \
+ Range* left_range = new Range( \
+ RangeBoundary::FromConstant(l_min), \
+ RangeBoundary::FromConstant(l_max)); \
+ Range* shift_range = new Range( \
+ RangeBoundary::FromConstant(r_min), \
+ RangeBoundary::FromConstant(r_max)); \
+ Op(left_range, shift_range, &min, &max); \
+ EXPECT(min.Equals(result_min)); \
+ if (!min.Equals(result_min)) OS::Print("%s\n", min.ToCString()); \
+ EXPECT(max.Equals(result_max)); \
+ if (!max.Equals(result_max)) OS::Print("%s\n", max.ToCString()); \
+ }
+
+ TEST_RANGE_OP(Range::Shl, -15, 100, 0, 2,
+ RangeBoundary(-60), RangeBoundary(400));
+ TEST_RANGE_OP(Range::Shl, -15, 100, -2, 2,
+ RangeBoundary(-60), RangeBoundary(400));
+ TEST_RANGE_OP(Range::Shl, -15, -10, 1, 2,
+ RangeBoundary(-60), RangeBoundary(-20));
+ TEST_RANGE_OP(Range::Shl, 5, 10, -2, 2,
+ RangeBoundary(5), RangeBoundary(40));
+ TEST_RANGE_OP(Range::Shl, -15, 100, 0, 64,
+ RangeBoundary::NegativeInfinity(),
+ RangeBoundary::PositiveInfinity());
+ TEST_RANGE_OP(Range::Shl, -1, 1, 63, 63,
+ RangeBoundary::NegativeInfinity(),
+ RangeBoundary::PositiveInfinity());
+ if (kBitsPerWord == 64) {
+ TEST_RANGE_OP(Range::Shl, -1, 1, 62, 62,
+ RangeBoundary(kSmiMin),
+ RangeBoundary::PositiveInfinity());
+ TEST_RANGE_OP(Range::Shl, -1, 1, 30, 30,
+ RangeBoundary(-1 << 30),
+ RangeBoundary(1 << 30));
+ } else {
+ TEST_RANGE_OP(Range::Shl, -1, 1, 30, 30,
+ RangeBoundary(kSmiMin),
+ RangeBoundary::PositiveInfinity());
+ TEST_RANGE_OP(Range::Shl, -1, 1, 62, 62,
+ RangeBoundary::NegativeInfinity(),
+ RangeBoundary::PositiveInfinity());
+ }
+ TEST_RANGE_OP(Range::Shl, 0, 100, 0, 64,
+ RangeBoundary(0), RangeBoundary::PositiveInfinity());
+ TEST_RANGE_OP(Range::Shl, -100, 0, 0, 64,
+ RangeBoundary::NegativeInfinity(), RangeBoundary(0));
+
+#undef TEST_RANGE_SHL
Vyacheslav Egorov (Google) 2014/06/12 15:46:17 TEST_RANGE_OP
Florian Schneider 2014/06/13 07:12:21 Done.
}
« no previous file with comments | « runtime/vm/intermediate_language.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698