<div dir="ltr">review:<div><a href="http://reviews.llvm.org/D17765">http://reviews.llvm.org/D17765</a><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-03-01 15:12 GMT+01:00 Piotr Padlewski <span dir="ltr"><<a href="mailto:piotr.padlewski@gmail.com" target="_blank">piotr.padlewski@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">my pass works for things like<div>(note that for test purposes I use lexical_cast that doesn't do anything, so the second one compiles)</div><div><br></div><div>int a = boost::lexical_cast<int>("42");</div><div>long long* b = boost::lexical_cast<long long*>("42");</div><div><br></div><div>=></div><div>auto a = boost::lexical_cast<int>("42");</div><div><div>auto b = boost::lexical_cast<long long*>("42");</div></div><div><br></div><div>but it doesnt work for</div><div><div>long long a = boost::lexical_cast<long long>("42");</div><div>=></div><div><div>auto long a = boost::lexical_cast<long long>("42");</div></div><div><br></div><div>I will post review in few minutes.</div><div><br></div><div></div></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2016-03-01 15:09 GMT+01:00 Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">+cfe-commits for wider audience.</div><div class="gmail_quote"><br></div><div class="gmail_quote">Hi Piotr,</div><div class="gmail_quote"><br></div><div class="gmail_quote"><span>On Tue, Mar 1, 2016 at 2:36 PM, Piotr Padlewski <span dir="ltr"><<a href="mailto:piotr.padlewski@gmail.com" target="_blank">piotr.padlewski@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi Alexey,</div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>I am currently writing new check</div><div><a href="https://llvm.org/bugs/show_bug.cgi?id=26763" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=26763</a></div></div></blockquote><div><br></div></span><div>Awesome! Sorry I didn't have time yet to comment on the issues you filed, though I have a couple of thoughts.</div><span><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>I have problem with SourceTypeInfo. I am doing the same thing as modernize-use-auto (for new expression) does, but it doesn't work for non pointer types.</div></div></blockquote><div><br></div></span><div>If you point me to the code, it will be easier to answer your question. Currently I can only guess what you're talking about.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>I have checked, and my code works for pointers types like long long.</div></div></blockquote><div><br></div></span><div>What do you mean by `long long` being a `pointer type`?</div><div><br></div><div>In any case, without seeing the code it's hard to answer your question. You can upload a patch to <a href="http://reviews.llvm.org" target="_blank">reviews.llvm.org</a> and add a test case that fails for you (or another good way to reproduce the issue you're facing).</div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Am I doing something wrong, or is it just bug in SourceTypeInfo.getEnd()?</div><span><font color="#888888"><div><br></div><div>Piotr</div></font></span></div>
</blockquote></span></div><span><font color="#888888"><br></font></span></div><span><font color="#888888"><div class="gmail_extra">-- Alexander</div></font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>