<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Feb 27, 2016 at 3:21 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 02/27/2016 08:16 AM, Hongbin Zheng wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    +  ///<br>
    +  /// @param BasePtr The base pointer of the ScopArrayInfo object<br>
    to invalidate.<br>
    +  /// @param Kind    The Kind of the ScopArrayInfo object.<br>
    +  void invalidateScopArrayInfo(Value *BasePtr,<br>
    ScopArrayInfo::MemoryKind Kind) {<br>
    +    ScopArrayInfoMap.erase(std::make_pair(BasePtr, Kind));<br>
<br>
Can we use a PointerIntPair as the key here instead? the BasePtr should<br>
be already guarded by the AssertingVH inside ScopArrayInfo.<br>
</blockquote>
<br></span>
What would be the benefit of using a PointerIntPair? This is a pure performance optimization, right?<br>
<br>
In case this is just about performance, I would prefer not to use PointerIntPairs. This code is likely not performance sensitive and PointerIntPairs are from my point of view harder to read and debug as normal pairs.<br></blockquote><div>Ok </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Best,<br>
Tobias<br></blockquote><div> </div></div><br></div></div>