Commit 3e809a61 authored by jkummerow's avatar jkummerow Committed by Commit bot

Fix compiler warnings on "make android_arm"

Review-Url: https://codereview.chromium.org/2264283007
Cr-Commit-Position: refs/heads/master@{#38937}
parent 71f93e60
...@@ -378,6 +378,9 @@ ...@@ -378,6 +378,9 @@
'arm_version%': '<(arm_version)', 'arm_version%': '<(arm_version)',
'host_os%': '<(host_os)', 'host_os%': '<(host_os)',
# Print to stdout on Android.
'v8_android_log_stdout%': 1,
'conditions': [ 'conditions': [
['android_ndk_root==""', { ['android_ndk_root==""', {
'variables': { 'variables': {
......
...@@ -376,7 +376,7 @@ const CreateLiteralParameters& CreateLiteralParametersOf(const Operator* op) { ...@@ -376,7 +376,7 @@ const CreateLiteralParameters& CreateLiteralParametersOf(const Operator* op) {
return OpParameter<CreateLiteralParameters>(op); return OpParameter<CreateLiteralParameters>(op);
} }
const BinaryOperationHint BinaryOperationHintOf(const Operator* op) { BinaryOperationHint BinaryOperationHintOf(const Operator* op) {
DCHECK(op->opcode() == IrOpcode::kJSBitwiseOr || DCHECK(op->opcode() == IrOpcode::kJSBitwiseOr ||
op->opcode() == IrOpcode::kJSBitwiseXor || op->opcode() == IrOpcode::kJSBitwiseXor ||
op->opcode() == IrOpcode::kJSBitwiseAnd || op->opcode() == IrOpcode::kJSBitwiseAnd ||
...@@ -391,7 +391,7 @@ const BinaryOperationHint BinaryOperationHintOf(const Operator* op) { ...@@ -391,7 +391,7 @@ const BinaryOperationHint BinaryOperationHintOf(const Operator* op) {
return OpParameter<BinaryOperationHint>(op); return OpParameter<BinaryOperationHint>(op);
} }
const CompareOperationHint CompareOperationHintOf(const Operator* op) { CompareOperationHint CompareOperationHintOf(const Operator* op) {
DCHECK(op->opcode() == IrOpcode::kJSEqual || DCHECK(op->opcode() == IrOpcode::kJSEqual ||
op->opcode() == IrOpcode::kJSNotEqual || op->opcode() == IrOpcode::kJSNotEqual ||
op->opcode() == IrOpcode::kJSStrictEqual || op->opcode() == IrOpcode::kJSStrictEqual ||
......
...@@ -374,9 +374,9 @@ std::ostream& operator<<(std::ostream&, CreateLiteralParameters const&); ...@@ -374,9 +374,9 @@ std::ostream& operator<<(std::ostream&, CreateLiteralParameters const&);
const CreateLiteralParameters& CreateLiteralParametersOf(const Operator* op); const CreateLiteralParameters& CreateLiteralParametersOf(const Operator* op);
const BinaryOperationHint BinaryOperationHintOf(const Operator* op); BinaryOperationHint BinaryOperationHintOf(const Operator* op);
const CompareOperationHint CompareOperationHintOf(const Operator* op); CompareOperationHint CompareOperationHintOf(const Operator* op);
// Interface for building JavaScript-level operators, e.g. directly from the // Interface for building JavaScript-level operators, e.g. directly from the
// AST. Most operators have no parameters, thus can be globally shared for all // AST. Most operators have no parameters, thus can be globally shared for all
......
...@@ -23,7 +23,7 @@ size_t hash_value(DeoptimizeReason reason) { ...@@ -23,7 +23,7 @@ size_t hash_value(DeoptimizeReason reason) {
return static_cast<uint8_t>(reason); return static_cast<uint8_t>(reason);
} }
char const* const DeoptimizeReasonToString(DeoptimizeReason reason) { char const* DeoptimizeReasonToString(DeoptimizeReason reason) {
static char const* kDeoptimizeReasonStrings[] = { static char const* kDeoptimizeReasonStrings[] = {
#define DEOPTIMIZE_REASON(Name, message) message, #define DEOPTIMIZE_REASON(Name, message) message,
DEOPTIMIZE_REASON_LIST(DEOPTIMIZE_REASON) DEOPTIMIZE_REASON_LIST(DEOPTIMIZE_REASON)
......
...@@ -90,7 +90,7 @@ std::ostream& operator<<(std::ostream&, DeoptimizeReason); ...@@ -90,7 +90,7 @@ std::ostream& operator<<(std::ostream&, DeoptimizeReason);
size_t hash_value(DeoptimizeReason reason); size_t hash_value(DeoptimizeReason reason);
char const* const DeoptimizeReasonToString(DeoptimizeReason reason); char const* DeoptimizeReasonToString(DeoptimizeReason reason);
} // namespace internal } // namespace internal
} // namespace v8 } // namespace v8
......
...@@ -912,7 +912,8 @@ inline bool IsDynamicVariableMode(VariableMode mode) { ...@@ -912,7 +912,8 @@ inline bool IsDynamicVariableMode(VariableMode mode) {
inline bool IsDeclaredVariableMode(VariableMode mode) { inline bool IsDeclaredVariableMode(VariableMode mode) {
return mode >= VAR && mode <= CONST; STATIC_ASSERT(VAR == 0); // Implies that mode >= VAR.
return mode <= CONST;
} }
......
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