From: Quentin Colombet Date: Sat, 24 Sep 2016 04:54:03 +0000 (+0000) Subject: [RegisterBankInfo] Constify the member of the XXXMapping maps. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=9b473140dd595020218362c4c87884ee997bd731;p=llvm [RegisterBankInfo] Constify the member of the XXXMapping maps. This makes it obvious that items in those maps behave like statically created objects. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@282327 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h b/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h index 0d9df7a6f55..757c4c44365 100644 --- a/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h +++ b/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h @@ -361,11 +361,11 @@ protected: /// Keep dynamically allocated PartialMapping in a separate map. /// This shouldn't be needed when everything gets TableGen'ed. - mutable DenseMap MapOfPartialMappings; + mutable DenseMap MapOfPartialMappings; /// Keep dynamically allocated ValueMapping in a separate map. /// This shouldn't be needed when everything gets TableGen'ed. - mutable DenseMap MapOfValueMappings; + mutable DenseMap MapOfValueMappings; /// Create a RegisterBankInfo that can accomodate up to \p NumRegBanks /// RegisterBank instances. diff --git a/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp b/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp index 52963caa8bf..bac9ba860bf 100644 --- a/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp +++ b/lib/CodeGen/GlobalISel/RegisterBankInfo.cpp @@ -344,7 +344,7 @@ RegisterBankInfo::getPartialMapping(unsigned StartIdx, unsigned Length, ++NumPartialMappingsCreated; - PartialMapping *&PartMapping = MapOfPartialMappings[Hash]; + const PartialMapping *&PartMapping = MapOfPartialMappings[Hash]; PartMapping = new PartialMapping{StartIdx, Length, RegBank}; return *PartMapping; } @@ -376,7 +376,7 @@ RegisterBankInfo::getValueMapping(const PartialMapping *BreakDown, ++NumValueMappingsCreated; - ValueMapping *&ValMapping = MapOfValueMappings[Hash]; + const ValueMapping *&ValMapping = MapOfValueMappings[Hash]; ValMapping = new ValueMapping{BreakDown, NumBreakDowns}; return *ValMapping; }