<div dir="ltr">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)</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Oct 20, 2016 at 6:47 AM Aaron Ballman <<a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aaron.ballman accepted this revision.<br class="gmail_msg">
aaron.ballman added a reviewer: aaron.ballman.<br class="gmail_msg">
aaron.ballman added a comment.<br class="gmail_msg">
This revision is now accepted and ready to land.<br class="gmail_msg">
<br class="gmail_msg">
LGTM<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: lib/Sema/SemaExprCXX.cpp:6287<br class="gmail_msg">
+            Context.hasSameUnqualifiedType(DestructedType,<br class="gmail_msg">
+                                           ObjectType->getPointeeType())) {<br class="gmail_msg">
+          Diag(OpLoc, diag::err_typecheck_member_reference_suggestion)<br class="gmail_msg">
----------------<br class="gmail_msg">
You can elide the curly braces.<br class="gmail_msg">
<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">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>