[Preamble] Check system dependencies in preamble too
authorIlya Biryukov <ibiryukov@google.com>
Mon, 9 Jul 2018 09:07:01 +0000 (09:07 +0000)
committerIlya Biryukov <ibiryukov@google.com>
Mon, 9 Jul 2018 09:07:01 +0000 (09:07 +0000)
commite5801b02e49b6fc4dd07702652ea70c9ea0275f3
treefdaeacb9e138bd577953945ce1c7553e86ef81c7
parent078c879a50f187c4c25e208565c7b34f1b3fc822
[Preamble] Check system dependencies in preamble too

Summary:
PrecompiledPreamble hasn't checked if the system dependencies changed
before. This resulted in invalid preamble not being rebuilt if headers
that changed were found in -isystem include paths.

This pattern is sometimes used to avoid showing warnings in third
party code, so we want to correctly handle those cases.

Tested in clangd, see the follow-up patch.

Reviewers: sammccall, ioeric

Reviewed By: sammccall

Subscribers: omtcyfz, cfe-commits

Differential Revision: https://reviews.llvm.org/D48946

llvm-svn: 336528
clang/lib/Frontend/PrecompiledPreamble.cpp