<div dir="ltr">Answered on the review thread. The problem is with BuiltinTypeLoc not storing the location for the end token of builtin type names consisting of multiple tokens. This can be worked around by re-lexing a fragment of the code for now.<div><br></div><div>Richard, does it make sense to store the end location in BuiltinTypeLoc?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 1, 2016 at 3:30 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">review:<div><a href="http://reviews.llvm.org/D17765" target="_blank">http://reviews.llvm.org/D17765</a><br></div></div><div class="HOEnZb"><div class="h5"><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><div><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>
</div></div></blockquote></div><br></div>