ilya-biryukov accepted this revision.
ilya-biryukov added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D48903#1187605, @simark wrote:

> In https://reviews.llvm.org/D48903#1187596, @ilya-biryukov wrote:
>
> > This revision got 'reopened' and is now in the list of accepted revision. 
> > Should we close it again?
>
>
> It got reverted a second time because it was breaking a test on Windows.  The 
> new bit is the change to `test/PCH/case-insensitive-include.c`, so it would 
> need review again.  If somebody else could run the tests on Windows, it would 
> make me a bit more confident too.


I see, thanks for the explanation.

LGTM for the update revision, given we have confirmation the tests pass on 
Windows.


Repository:
  rC Clang

https://reviews.llvm.org/D48903



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to