Thanks for looking into this. I've reverted the test assertions on the
branch in r352354.
On Mon, Jan 28, 2019 at 1:14 AM Sam McCall wrote:
>
> This is kind of expected: r347205 was a bugfix.
> Reverting it causes a real bug, and the tests are failing. The bug is
> acceptable for 8.0 - it affect
This is kind of expected: r347205 was a bugfix.
Reverting it causes a real bug, and the tests are failing. The bug is
acceptable for 8.0 - it affects only a marginal case (go-to-definition on a
#include *outside* the preamble section of a file that was also
transitively #included in the preamble).
Sorry to bring more bad news, but merging this seems to have regressed
a clangd test on the branch (I didn't notice at the time, because I
ran the wrong set of tests, d'oh).
I've searched my inbox but couldn't find any recent commits touching
the test. Do you have any idea what might be happening?
Merged to 8.0 in r352225.
On Thu, Jan 24, 2019 at 10:55 AM Sam McCall via cfe-commits
wrote:
>
> Author: sammccall
> Date: Thu Jan 24 10:55:24 2019
> New Revision: 352079
>
> URL: http://llvm.org/viewvc/llvm-project?rev=352079&view=rev
> Log:
> [FileManager] Revert r347205 to avoid PCH file-descr
