Address comments.

Author: kazutakahirata

llvm/include/llvm/ProfileData/MemProfReader.h

@@ -46,7 +46,8 @@ class MemProfReader {
     return Iterator(this);
   }
 
-  // Take the complete profile data.
+  // Take the complete profile data.  Once this function is invoked,
+  // MemProfReader no longer owns the MemProf profile.
   IndexedMemProfData takeMemProfData() { return std::move(MemProfData); }
 
   virtual Error
@@ -82,7 +83,7 @@ class MemProfReader {
   virtual ~MemProfReader() = default;
 
   // Initialize the MemProfReader with the given MemProf profile.
-  MemProfReader(IndexedMemProfData MemProfData)
+  MemProfReader(IndexedMemProfData &&MemProfData)
       : MemProfData(std::move(MemProfData)) {}
 
 protected:

llvm/unittests/ProfileData/MemProfTest.cpp

@@ -440,7 +440,7 @@ TEST(MemProf, BaseMemProfReader) {
   FakeRecord.AllocSites.emplace_back(/*CSId=*/CSId, /*MB=*/Block);
   MemProfData.Records.insert({F1.hash(), FakeRecord});
 
-  MemProfReader Reader(MemProfData);
+  MemProfReader Reader(std::move(MemProfData));
 
   llvm::SmallVector<MemProfRecord, 1> Records;
   for (const auto &KeyRecordPair : Reader) {
@@ -478,7 +478,7 @@ TEST(MemProf, BaseMemProfReaderWithCSIdMap) {
       /*MB=*/Block);
   MemProfData.Records.insert({F1.hash(), FakeRecord});
 
-  MemProfReader Reader(MemProfData);
+  MemProfReader Reader(std::move(MemProfData));
 
   llvm::SmallVector<MemProfRecord, 1> Records;
   for (const auto &KeyRecordPair : Reader) {