simpleperf: remove RecordCache.

When using debug-unwind cmd for system wide profiling result, I
found it might took a lot of memory because RecordCache cached
too many samples. Since simpleperf no longer relies on RecordCache
to sort records, I think it is fine to remove RecordCache instead
of fixing it.

Bug: none.
Test: run simpleperf_unit_test.
Change-Id: Ie28ce17b4158add455004a56bbdac745f9d05f19

1 files changed, 2 insertions, 4 deletions

diff --git a/simpleperf/record_file.h b/simpleperf/record_file.h
index 4ec6c31c..185862a8 100644
--- a/simpleperf/record_file.h
+++ b/simpleperf/record_file.h
@@ -130,13 +130,12 @@ class RecordFileReader {
   // is by calling ReadRecord() in a loop.
 
   // If sorted is true, sort records before passing them to callback function.
-  bool ReadDataSection(const std::function<bool(std::unique_ptr<Record>)>& callback,
-                       bool sorted = true);
+  bool ReadDataSection(const std::function<bool(std::unique_ptr<Record>)>& callback);
 
   // Read next record. If read successfully, set [record] and return true.
   // If there is no more records, set [record] to nullptr and return true.
   // Otherwise return false.
-  bool ReadRecord(std::unique_ptr<Record>& record, bool sorted = true);
+  bool ReadRecord(std::unique_ptr<Record>& record);
 
   size_t GetAttrIndexOfRecord(const Record* record);
 
@@ -183,7 +182,6 @@ class RecordFileReader {
   size_t event_id_pos_in_sample_records_;
   size_t event_id_reverse_pos_in_non_sample_records_;
 
-  std::unique_ptr<RecordCache> record_cache_;
   uint64_t read_record_size_;
 
   DISALLOW_COPY_AND_ASSIGN(RecordFileReader);