summaryrefslogtreecommitdiff
path: root/compiler/optimizing/optimizing_unit_test.h
diff options
context:
space:
mode:
author Santiago Aboy Solanes <solanes@google.com> 2023-02-14 14:03:38 +0000
committer Santiago Aboy Solanes <solanes@google.com> 2023-02-15 14:07:32 +0000
commitb22e708890b61e6a82bb8a18e98413c2bd4933c9 (patch)
treec118b74fa4a64889eb59a0844b0384500901bae4 /compiler/optimizing/optimizing_unit_test.h
parent8cc1c0fc045623b86cd35e93deb23004b09a3d69 (diff)
Remove RTI validation from our checks
It is possible to have untyped RTI in valid graphs due to e.g. dead code, redundant phis not replaced yet. By removing this validation, we shift the burden of the validation to the user of the RTI. Bug: 252799892 Fixes: 252799892 Test: art/test/testrunner/testrunner.py --host --64 --optimizing -b Test: dex2oat compiling the apps in the bug Change-Id: I112497bf10c180b651fd5b6e268799bc3abb27b3
Diffstat (limited to 'compiler/optimizing/optimizing_unit_test.h')
-rw-r--r--compiler/optimizing/optimizing_unit_test.h18
1 files changed, 1 insertions, 17 deletions
diff --git a/compiler/optimizing/optimizing_unit_test.h b/compiler/optimizing/optimizing_unit_test.h
index e62ccf0f3a..2ebaf58d65 100644
--- a/compiler/optimizing/optimizing_unit_test.h
+++ b/compiler/optimizing/optimizing_unit_test.h
@@ -321,25 +321,10 @@ class OptimizingUnitTestHelper {
// Run GraphChecker with all checks.
//
// Return: the status whether the run is successful.
- bool CheckGraph(HGraph* graph, std::ostream& oss = std::cerr) {
- return CheckGraph(graph, /*check_ref_type_info=*/true, oss);
- }
-
bool CheckGraph(std::ostream& oss = std::cerr) {
return CheckGraph(graph_, oss);
}
- // Run GraphChecker with all checks except reference type information checks.
- //
- // Return: the status whether the run is successful.
- bool CheckGraphSkipRefTypeInfoChecks(HGraph* graph, std::ostream& oss = std::cerr) {
- return CheckGraph(graph, /*check_ref_type_info=*/false, oss);
- }
-
- bool CheckGraphSkipRefTypeInfoChecks(std::ostream& oss = std::cerr) {
- return CheckGraphSkipRefTypeInfoChecks(graph_, oss);
- }
-
HEnvironment* ManuallyBuildEnvFor(HInstruction* instruction,
ArenaVector<HInstruction*>* current_locals) {
HEnvironment* environment = new (GetAllocator()) HEnvironment(
@@ -532,9 +517,8 @@ class OptimizingUnitTestHelper {
}
protected:
- bool CheckGraph(HGraph* graph, bool check_ref_type_info, std::ostream& oss) {
+ bool CheckGraph(HGraph* graph, std::ostream& oss) {
GraphChecker checker(graph);
- checker.SetRefTypeInfoCheckEnabled(check_ref_type_info);
checker.Run();
checker.Dump(oss);
return checker.IsValid();