<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jan 3, 2015 at 5:25 AM, 彩云追月 <span dir="ltr"><<a href="mailto:wuming_81@163.com" target="_blank">wuming_81@163.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div>   Hello, everyone.</div><div>    1) I found someone uses function  StringMap::GetOrCreateValue  many times in clang. But  it does exist in  class StringMap.So I added tow overloaded functions of this name.</div></div></blockquote><div><br>These were removed a couple of months ago in r222306.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div>    2) the return value type of function StringSet::insert, is as bellow:</div><div>        <i>std::pair<typename base::iterator, bool></i></div><div>        </div><div>        I have an old versioned llvm/clang code.  this return value type is bool</div><div>        I think the current status is right.But some calls of this function is the clang  has  not been modified, i.e the codes are still like:</div><div>    if (some_string_set.insert( ... ))</div><div>        I have modified some places.</div></div></blockquote><div><br>The right return type for a map insert is pair<iterator, bool> to be compatible with STL containers/concepts.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div>    3) similar problems exist with function SmallPointerSet::insert()</div><div>        but ,so many many places to modify.</div><div>        I have only modified a few places.Sorry!</div></div></blockquote><div><br>If you sync up your LLVM and Clang projects you should find they build without any need for further modification.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:Arial"><div><br></div></div><br><br><span title="neteasefooter"><span></span></span><br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div><br></div></div>