<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 21, 2016 at 3:16 AM Alex Lorenz <<a href="mailto:arphaman@gmail.com">arphaman@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">arphaman updated this revision to Diff 75403.<br class="gmail_msg">
arphaman added a comment.<br class="gmail_msg">
<br class="gmail_msg">
The updated patch improves error handling and adds a test for the fixit.<br class="gmail_msg">
<br class="gmail_msg">
> If we issue a fixit we should recover as-if the code was written with the fixit in. Does this code do that? (can we test it? I know we test some fixits - not sure it's necessary/worthwhile to test them all, but maybe we have a good idiom for testing that the recovery is correct)<br class="gmail_msg">
<br class="gmail_msg">
This code does perform recovery, but the constructed AST for the destructor calls is different from the AST that would have been constructed if the code was correct: we still end up building the pseudo destructor expression. I'm not sure how important is that though, so please let me know if I should try and make the ASTs the same.<br class="gmail_msg"></blockquote><div><br>I imagine we need the matching AST, but don't know for sure/how the difference might result in different behavior downstream (CC'd Mr. Smith in case he can quickly provide an opinion here)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
<br class="gmail_msg">
Repository:<br class="gmail_msg">
  rL LLVM<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D25817" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25817</a><br class="gmail_msg">
<br class="gmail_msg">
Files:<br class="gmail_msg">
  lib/Sema/SemaExprCXX.cpp<br class="gmail_msg">
  test/CXX/special/class.dtor/p10-0x.cpp<br class="gmail_msg">
  test/FixIt/fixit.cpp<br class="gmail_msg">
  test/SemaCXX/pseudo-destructors.cpp<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>