OLD | NEW |
1 // Copyright 2013 the V8 project authors. All rights reserved. | 1 // Copyright 2013 the V8 project authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "src/hydrogen-uint32-analysis.h" | 5 #include "src/hydrogen-uint32-analysis.h" |
6 | 6 |
7 namespace v8 { | 7 namespace v8 { |
8 namespace internal { | 8 namespace internal { |
9 | 9 |
10 | 10 |
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
55 // operation. | 55 // operation. |
56 if (store->value() == val) { | 56 if (store->value() == val) { |
57 // Clamping or a conversion to double should have beed inserted. | 57 // Clamping or a conversion to double should have beed inserted. |
58 ASSERT(store->elements_kind() != EXTERNAL_UINT8_CLAMPED_ELEMENTS); | 58 ASSERT(store->elements_kind() != EXTERNAL_UINT8_CLAMPED_ELEMENTS); |
59 ASSERT(store->elements_kind() != EXTERNAL_FLOAT32_ELEMENTS); | 59 ASSERT(store->elements_kind() != EXTERNAL_FLOAT32_ELEMENTS); |
60 ASSERT(store->elements_kind() != EXTERNAL_FLOAT64_ELEMENTS); | 60 ASSERT(store->elements_kind() != EXTERNAL_FLOAT64_ELEMENTS); |
61 return true; | 61 return true; |
62 } | 62 } |
63 } | 63 } |
64 } else if (use->IsCompareNumericAndBranch()) { | 64 } else if (use->IsCompareNumericAndBranch()) { |
| 65 return false; // TODO(svenpanne/3380): Fix and re-enable! |
65 HCompareNumericAndBranch* c = HCompareNumericAndBranch::cast(use); | 66 HCompareNumericAndBranch* c = HCompareNumericAndBranch::cast(use); |
66 return IsUint32Operation(c->left()) && IsUint32Operation(c->right()); | 67 return IsUint32Operation(c->left()) && IsUint32Operation(c->right()); |
67 } | 68 } |
68 | 69 |
69 return false; | 70 return false; |
70 } | 71 } |
71 | 72 |
72 | 73 |
73 // Iterate over all uses and verify that they are uint32 safe: either don't | 74 // Iterate over all uses and verify that they are uint32 safe: either don't |
74 // distinguish between int32 and uint32 due to their bitwise nature or | 75 // distinguish between int32 and uint32 due to their bitwise nature or |
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
231 | 232 |
232 // Some phis might have been optimistically marked with kUint32 flag. | 233 // Some phis might have been optimistically marked with kUint32 flag. |
233 // Remove this flag from those phis that are unsafe and propagate | 234 // Remove this flag from those phis that are unsafe and propagate |
234 // this information transitively potentially clearing kUint32 flag | 235 // this information transitively potentially clearing kUint32 flag |
235 // from some non-phi operations that are used as operands to unsafe phis. | 236 // from some non-phi operations that are used as operands to unsafe phis. |
236 UnmarkUnsafePhis(); | 237 UnmarkUnsafePhis(); |
237 } | 238 } |
238 | 239 |
239 | 240 |
240 } } // namespace v8::internal | 241 } } // namespace v8::internal |
OLD | NEW |