<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 5, 2018, 13:58 Ilya Biryukov via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ilya-biryukov added a comment.<br>
<br>
> There's the usual concern that the current behavior is reasonable and people like it.<br>
<br>
I think it would be reasonable to say that a large portion of C++ users are used to the behavior that this patch introduces.<br>
Specifically, all IDEs (Clion, Eclipse, Visual Studio for certain, not sure about XCode) will see the contents of the headers that the user typed and not the other way around.<br>
It does not cause any inconsistencies there, the IDE will save all your open files when you hit "build". That also means reading files from the filesystem gives non-useful errors in that case, because the build will actually see the files from the editors.<br></blockquote></div></div><div dir="auto">I agree, which is why I described both behaviors as reasonable :-)</div><div dir="auto">I don't this this address either of my concerns, though.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Repository:<br>
  rCTE Clang Tools Extra<br>
<br>
<a href="https://reviews.llvm.org/D54077" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D54077</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>