From be727cf341cc6fdc90ddb168c06d8c67230d834f Mon Sep 17 00:00:00 2001 From: Vladimir Marko Date: Tue, 6 Feb 2024 15:59:21 +0000 Subject: Remove `HInstruction::IsFieldAccess()`. Test: m test-art-host-gtest Test: testrunner.py --host --optimizing Change-Id: I697263c647d76a13b9de56a2134a6e8554c20929 --- compiler/optimizing/nodes.h | 10 ---------- 1 file changed, 10 deletions(-) (limited to 'compiler/optimizing/nodes.h') diff --git a/compiler/optimizing/nodes.h b/compiler/optimizing/nodes.h index bcf27ae9fa..66bef7a8f1 100644 --- a/compiler/optimizing/nodes.h +++ b/compiler/optimizing/nodes.h @@ -2442,13 +2442,7 @@ class HInstruction : public ArenaObject { UNREACHABLE(); } - virtual bool IsFieldAccess() const { - return false; - } - virtual const FieldInfo& GetFieldInfo() const { - CHECK(IsFieldAccess()) << "Only callable on field accessors not " << DebugName() << " " - << *this; LOG(FATAL) << "Must be overridden by field accessors. Not implemented by " << *this; UNREACHABLE(); } @@ -6105,7 +6099,6 @@ class HInstanceFieldGet final : public HExpression<1> { return (HInstruction::ComputeHashCode() << 7) | GetFieldOffset().SizeValue(); } - bool IsFieldAccess() const override { return true; } const FieldInfo& GetFieldInfo() const override { return field_info_; } MemberOffset GetFieldOffset() const { return field_info_.GetFieldOffset(); } DataType::Type GetFieldType() const { return field_info_.GetFieldType(); } @@ -6179,7 +6172,6 @@ class HInstanceFieldSet final : public HExpression<2> { return (obj == InputAt(0)) && art::CanDoImplicitNullCheckOn(GetFieldOffset().Uint32Value()); } - bool IsFieldAccess() const override { return true; } const FieldInfo& GetFieldInfo() const override { return field_info_; } MemberOffset GetFieldOffset() const { return field_info_.GetFieldOffset(); } DataType::Type GetFieldType() const { return field_info_.GetFieldType(); } @@ -7252,7 +7244,6 @@ class HStaticFieldGet final : public HExpression<1> { return (HInstruction::ComputeHashCode() << 7) | GetFieldOffset().SizeValue(); } - bool IsFieldAccess() const override { return true; } const FieldInfo& GetFieldInfo() const override { return field_info_; } MemberOffset GetFieldOffset() const { return field_info_.GetFieldOffset(); } DataType::Type GetFieldType() const { return field_info_.GetFieldType(); } @@ -7306,7 +7297,6 @@ class HStaticFieldSet final : public HExpression<2> { } bool IsClonable() const override { return true; } - bool IsFieldAccess() const override { return true; } const FieldInfo& GetFieldInfo() const override { return field_info_; } MemberOffset GetFieldOffset() const { return field_info_.GetFieldOffset(); } DataType::Type GetFieldType() const { return field_info_.GetFieldType(); } -- cgit v1.2.3-59-g8ed1b