| 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! | |
| 66 HCompareNumericAndBranch* c = HCompareNumericAndBranch::cast(use); | 65 HCompareNumericAndBranch* c = HCompareNumericAndBranch::cast(use); |
| 67 return IsUint32Operation(c->left()) && IsUint32Operation(c->right()); | 66 return IsUint32Operation(c->left()) && IsUint32Operation(c->right()); |
| 68 } | 67 } |
| 69 | 68 |
| 70 return false; | 69 return false; |
| 71 } | 70 } |
| 72 | 71 |
| 73 | 72 |
| 74 // Iterate over all uses and verify that they are uint32 safe: either don't | 73 // Iterate over all uses and verify that they are uint32 safe: either don't |
| 75 // distinguish between int32 and uint32 due to their bitwise nature or | 74 // distinguish between int32 and uint32 due to their bitwise nature or |
| (...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 232 | 231 |
| 233 // Some phis might have been optimistically marked with kUint32 flag. | 232 // Some phis might have been optimistically marked with kUint32 flag. |
| 234 // Remove this flag from those phis that are unsafe and propagate | 233 // Remove this flag from those phis that are unsafe and propagate |
| 235 // this information transitively potentially clearing kUint32 flag | 234 // this information transitively potentially clearing kUint32 flag |
| 236 // from some non-phi operations that are used as operands to unsafe phis. | 235 // from some non-phi operations that are used as operands to unsafe phis. |
| 237 UnmarkUnsafePhis(); | 236 UnmarkUnsafePhis(); |
| 238 } | 237 } |
| 239 | 238 |
| 240 | 239 |
| 241 } } // namespace v8::internal | 240 } } // namespace v8::internal |
| OLD | NEW |