diff options
| author | 2020-11-12 17:05:28 +0000 | |
|---|---|---|
| committer | 2020-11-13 10:07:21 +0000 | |
| commit | b6837f0350ff66c13582b0e94178dd5ca283ff0a (patch) | |
| tree | f79fff81352545efe967850e3d17e32255dcfecd /compiler/optimizing/scheduler_test.cc | |
| parent | 32c2eb81320f24b5bab24754204b8be95faa08b0 (diff) | |
Revert^2 "Partial LSE analysis & store removal"
A ScopedArenaAllocator in a single test was accidentally loaded using
operator new which is not supported. This caused a memory leak.
This reverts commit fe270426c8a2a69a8f669339e83b86fbf40e25a1.
This unreverts commit bb6cda60e4418c0ab557ea4090e046bed8206763.
Bug: 67037140
Reason for revert: Fixed memory leak in
LoadStoreAnalysisTest.PartialEscape test case
Test: SANITIZE_HOST=address ASAN_OPTIONS=detect_leaks=0 m test-art-host-gtest-dependencies
Run art_compiler_tests
Change-Id: I34fa2079df946ae54b8c91fa771a44d56438a719
Diffstat (limited to 'compiler/optimizing/scheduler_test.cc')
| -rw-r--r-- | compiler/optimizing/scheduler_test.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/compiler/optimizing/scheduler_test.cc b/compiler/optimizing/scheduler_test.cc index 94f1599e62..c166a46082 100644 --- a/compiler/optimizing/scheduler_test.cc +++ b/compiler/optimizing/scheduler_test.cc @@ -273,7 +273,8 @@ class SchedulerTest : public OptimizingUnitTest { entry->AddInstruction(instr); } - HeapLocationCollector heap_location_collector(graph_, GetScopedAllocator()); + HeapLocationCollector heap_location_collector( + graph_, GetScopedAllocator(), /*for_partial_elimination=*/false); heap_location_collector.VisitBasicBlock(entry); heap_location_collector.BuildAliasingMatrix(); TestSchedulingGraph scheduling_graph(GetScopedAllocator(), &heap_location_collector); |