r263344 - clang-cl: Add a test for the interaction of /Yc and /showIncludes.

Renato Golin via cfe-commits cfe-commits at lists.llvm.org
Sun Mar 13 01:26:00 PST 2016


On 13 March 2016 at 02:56, Nico Weber via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Author: nico
> Date: Sat Mar 12 13:55:59 2016
> New Revision: 263344
>
> URL: http://llvm.org/viewvc/llvm-project?rev=263344&view=rev
> Log:
> clang-cl: Add a test for the interaction of /Yc and /showIncludes.
>
> We almost get this right, but not completely (see FIXME).  It looks like /FI
> headers generally aren't included in /showIncludes yet, but they should be.
> But it seems good to have test coverage for the bits that already work.

Hi Nico,

This is segfaulting on all AArch64 bots:

http://lab.llvm.org:8011/builders/clang-cmake-aarch64-quick/builds/5433

http://lab.llvm.org:8011/builders/clang-cmake-aarch64-full/builds/1584

http://lab.llvm.org:8011/builders/clang-cmake-aarch64-42vma/builds/6668

And now that you have added more patches on the same area, it's not
easy to revert.

Can you please have a look?

Thanks!

--renato


More information about the cfe-commits mailing list