[PATCH] D103797: [clang] Use resolved path for headers in decluse diagnostics

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet created this revision.
kadircet requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

A header spelling in a source file could potentially mean multiple
sources, especially in projects with multiple include search paths. We
try to eliminate this ambiguity by using the resolved path in the
diagnostics. This should improve the life of both the developers and the
tooling.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D103797

Files:
  clang/lib/Lex/ModuleMap.cpp
  clang/test/Modules/declare-use1.cpp
  clang/test/Modules/declare-use2.cpp
  clang/test/Modules/declare-use3.cpp
  clang/test/Modules/header-attribs.cpp
  clang/test/Modules/strict-decluse-headers.cpp
  clang/test/Modules/strict-decluse.cpp
  clang/test/Modules/textual-headers.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103797.350216.patch
Type: text/x-patch
Size: 6030 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210607/f9cd9ade/attachment.bin>