<div dir="ltr">I think this should be reverted until we figure out how to solve it. Moreover, it looks expensive to check for each file if it's within a system path. I would really like to measure compile time for this change before it goes definitely in. </div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 3, 2016 at 4:53 PM, Justin Bogner via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Taewook Oh via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> writes:<br>
> Author: twoh<br>
> Date: Fri Jun 3 13:52:51 2016<br>
> New Revision: 271708<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=271708&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=271708&view=rev</a><br>
> Log:<br>
> Use the name of the file on disk to issue a new diagnostic about<br>
> non-portable #include and #import paths.<br>
><br>
> Differential Revision: <a href="http://reviews.llvm.org/D19843" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19843</a><br>
> Corresponding LLVM change: <a href="http://reviews.llvm.org/D19842" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19842</a><br>
><br>
> Patch by Eric Niebler<br>
><br>
><br>
> Added:<br>
> cfe/trunk/test/Lexer/Inputs/case-insensitive-include.h<br>
> cfe/trunk/test/Lexer/case-insensitive-include-ms.c<br>
> cfe/trunk/test/Lexer/case-insensitive-include.c<br>
> Modified:<br>
> cfe/trunk/include/clang/Basic/DiagnosticGroups.td<br>
> cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td<br>
> cfe/trunk/include/clang/Basic/FileManager.h<br>
> cfe/trunk/include/clang/Basic/VirtualFileSystem.h<br>
> cfe/trunk/include/clang/Lex/DirectoryLookup.h<br>
> cfe/trunk/include/clang/Lex/HeaderSearch.h<br>
> cfe/trunk/lib/Basic/FileManager.cpp<br>
> cfe/trunk/lib/Basic/VirtualFileSystem.cpp<br>
> cfe/trunk/lib/Lex/HeaderSearch.cpp<br>
> cfe/trunk/lib/Lex/PPDirectives.cpp<br>
> cfe/trunk/test/PCH/case-insensitive-include.c<br>
><br>
> Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td<br>
> URL:<br>
> <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
><br>
> --- cfe/trunk/include/clang/Basic/DiagnosticGroups.td (original)<br>
> +++ cfe/trunk/include/clang/Basic/DiagnosticGroups.td Fri Jun 3 13:52:51 2016<br>
> @@ -390,6 +390,7 @@ def : DiagGroup<"sequence-point", [Unseq<br>
> def AmbiguousMacro : DiagGroup<"ambiguous-macro">;<br>
> def KeywordAsMacro : DiagGroup<"keyword-macro">;<br>
> def ReservedIdAsMacro : DiagGroup<"reserved-id-macro">;<br>
> +def NonportableIncludePath : DiagGroup<"nonportable-include-path">;<br>
><br>
> // Just silence warnings about -Wstrict-aliasing for now.<br>
> def : DiagGroup<"strict-aliasing=0">;<br>
><br>
> Modified: cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td (original)<br>
> +++ cfe/trunk/include/clang/Basic/DiagnosticLexKinds.td Fri Jun 3 13:52:51 2016<br>
> @@ -274,6 +274,10 @@ def ext_missing_whitespace_after_macro_n<br>
> "whitespace required after macro name">;<br>
> def warn_missing_whitespace_after_macro_name : Warning<<br>
> "whitespace recommended after macro name">;<br>
> +def pp_nonportable_path : Warning<<br>
> + "non-portable path to file '%0'; specified path differs in case from file"<br>
> + " name on disk">,<br>
> + InGroup<NonportableIncludePath>;<br>
><br>
> def pp_pragma_once_in_main_file : Warning<"#pragma once in main file">,<br>
> InGroup<DiagGroup<"pragma-once-outside-header">>;<br>
><br>
> Modified: cfe/trunk/include/clang/Basic/FileManager.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/FileManager.h?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/FileManager.h?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/include/clang/Basic/FileManager.h (original)<br>
> +++ cfe/trunk/include/clang/Basic/FileManager.h Fri Jun 3 13:52:51 2016<br>
> @@ -52,6 +52,7 @@ public:<br>
> /// descriptor for the file.<br>
> class FileEntry {<br>
> const char *Name; // Name of the file.<br>
> + std::string RealPathName; // Real path to the file; could be empty.<br>
> off_t Size; // File size in bytes.<br>
> time_t ModTime; // Modification time of file.<br>
> const DirectoryEntry *Dir; // Directory file lives in.<br>
> @@ -82,6 +83,7 @@ public:<br>
> }<br>
><br>
> const char *getName() const { return Name; }<br>
> + StringRef tryGetRealPathName() const { return RealPathName; }<br>
> bool isValid() const { return IsValid; }<br>
> off_t getSize() const { return Size; }<br>
> unsigned getUID() const { return UID; }<br>
><br>
> Modified: cfe/trunk/include/clang/Basic/VirtualFileSystem.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/VirtualFileSystem.h?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/VirtualFileSystem.h?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/include/clang/Basic/VirtualFileSystem.h (original)<br>
> +++ cfe/trunk/include/clang/Basic/VirtualFileSystem.h Fri Jun 3 13:52:51 2016<br>
> @@ -91,6 +91,13 @@ public:<br>
> virtual ~File();<br>
> /// \brief Get the status of the file.<br>
> virtual llvm::ErrorOr<Status> status() = 0;<br>
> + /// \brief Get the name of the file<br>
> + virtual llvm::ErrorOr<StringRef> getName() {<br>
> + if (auto Status = status())<br>
> + return Status->getName();<br>
> + else<br>
> + return Status.getError();<br>
> + }<br>
> /// \brief Get the contents of the file as a \p MemoryBuffer.<br>
> virtual llvm::ErrorOr<std::unique_ptr<llvm::MemoryBuffer>><br>
> getBuffer(const Twine &Name, int64_t FileSize = -1,<br>
><br>
> Modified: cfe/trunk/include/clang/Lex/DirectoryLookup.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Lex/DirectoryLookup.h?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Lex/DirectoryLookup.h?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/include/clang/Lex/DirectoryLookup.h (original)<br>
> +++ cfe/trunk/include/clang/Lex/DirectoryLookup.h Fri Jun 3 13:52:51 2016<br>
> @@ -151,6 +151,9 @@ public:<br>
> ///<br>
> /// \param HS The header search instance to search with.<br>
> ///<br>
> + /// \param IncludeLoc the source location of the #include or #import<br>
> + /// directive.<br>
> + ///<br>
> /// \param SearchPath If not NULL, will be set to the search path relative<br>
> /// to which the file was found.<br>
> ///<br>
> @@ -172,6 +175,7 @@ public:<br>
> /// a framework include ("Foo.h" -> "Foo/Foo.h"), set the new name to this<br>
> /// vector and point Filename to it.<br>
> const FileEntry *LookupFile(StringRef &Filename, HeaderSearch &HS,<br>
> + SourceLocation IncludeLoc,<br>
> SmallVectorImpl<char> *SearchPath,<br>
> SmallVectorImpl<char> *RelativePath,<br>
> Module *RequestingModule,<br>
><br>
> Modified: cfe/trunk/include/clang/Lex/HeaderSearch.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Lex/HeaderSearch.h?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Lex/HeaderSearch.h?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/include/clang/Lex/HeaderSearch.h (original)<br>
> +++ cfe/trunk/include/clang/Lex/HeaderSearch.h Fri Jun 3 13:52:51 2016<br>
> @@ -580,8 +580,9 @@ private:<br>
> /// \brief Look up the file with the specified name and determine its owning<br>
> /// module.<br>
> const FileEntry *<br>
> - getFileAndSuggestModule(StringRef FileName, const DirectoryEntry *Dir,<br>
> - bool IsSystemHeaderDir, Module *RequestingModule,<br>
> + getFileAndSuggestModule(StringRef FileName, SourceLocation IncludeLoc,<br>
> + const DirectoryEntry *Dir, bool IsSystemHeaderDir,<br>
> + Module *RequestingModule,<br>
> ModuleMap::KnownHeader *SuggestedModule);<br>
><br>
> public:<br>
><br>
> Modified: cfe/trunk/lib/Basic/FileManager.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/FileManager.cpp?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/FileManager.cpp?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/lib/Basic/FileManager.cpp (original)<br>
> +++ cfe/trunk/lib/Basic/FileManager.cpp Fri Jun 3 13:52:51 2016<br>
> @@ -312,6 +312,9 @@ const FileEntry *FileManager::getFile(St<br>
> UFE.InPCH = Data.InPCH;<br>
> UFE.File = std::move(F);<br>
> UFE.IsValid = true;<br>
> + if (UFE.File)<br>
> + if (auto RealPathName = UFE.File->getName())<br>
> + UFE.RealPathName = RealPathName->str();<br>
<br>
</div></div>I'm getting complaints about this line under ASAN on a large number of<br>
tests locally. Here's the lit output from one of them:<br>
<br>
<br><br>
********************<br>
FAIL: Clang :: VFS/relative-path.c (9804 of 26380)<br>
******************** TEST 'Clang :: VFS/relative-path.c' FAILED ********************<br>
Script:<br>
--<br>
mkdir -p /Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.tmp<br>
cd /Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.tmp<br>
sed -e "s:INPUT_DIR:/Volumes/NewYork/bogner/code/llvm/tools/clang/test/VFS/Inputs:g" -e "s:OUT_DIR:/Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.tmp:g" /Volumes/NewYork/bogner/code/llvm/tools/clang/test/VFS/Inputs/vfsoverlay.yaml > /Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.tmp.yaml<br>
/Users/bogner/build/llvm-san/./bin/clang -cc1 -internal-isystem /Volumes/NewYork/bogner/build/llvm-san/bin/../lib/clang/3.9.0/include -nostdsysteminc -Werror -I . -ivfsoverlay /Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.tmp.yaml -fsyntax-only /Volumes/NewYork/bogner/code/llvm/tools/clang/test/VFS/relative-path.c<br>
--<br>
Exit Code: 134<br>
<br>
Command Output (stderr):<br>
--<br>
=================================================================<br>
==77391==ERROR: AddressSanitizer: heap-use-after-free on address 0x60700000a580 at pc 0x00011668488f bp 0x7fff5a4930f0 sp 0x7fff5a4928a8<br>
READ of size 77 at 0x60700000a580 thread T0<br>
#0 0x11668488e in wrap_memmove asan_interceptors.cc:445<br>
#1 0x7fff8ad36224 in std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >::__init(char const*, unsigned long) (libc++.1.dylib+0x3f224)<br>
#2 0x10576eb20 in llvm::StringRef::str() const string:2076<br>
#3 0x108dc4007 in clang::FileManager::getFile(llvm::StringRef, bool, bool) FileManager.cpp:317<br>
#4 0x10cbf5d53 in clang::HeaderSearch::getFileAndSuggestModule(llvm::StringRef, clang::SourceLocation, clang::DirectoryEntry const*, bool, clang::Module*, clang::ModuleMap::KnownHeader*) HeaderSearch.cpp:258<br>
#5 0x10cbf64d5 in clang::DirectoryLookup::LookupFile(llvm::StringRef&, clang::HeaderSearch&, clang::SourceLocation, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::Module*, clang::ModuleMap::KnownHeader*, bool&, bool&, llvm::SmallVectorImpl<char>&) const HeaderSearch.cpp:302<br>
#6 0x10cbf96d2 in clang::HeaderSearch::LookupFile(llvm::StringRef, clang::SourceLocation, bool, clang::DirectoryLookup const*, clang::DirectoryLookup const*&, llvm::ArrayRef<std::__1::pair<clang::FileEntry const*, clang::DirectoryEntry const*> >, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::Module*, clang::ModuleMap::KnownHeader*, bool, bool) HeaderSearch.cpp:717<br>
#7 0x10cca475e in clang::Preprocessor::LookupFile(clang::SourceLocation, llvm::StringRef, bool, clang::DirectoryLookup const*, clang::FileEntry const*, clang::DirectoryLookup const*&, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::ModuleMap::KnownHeader*, bool) PPDirectives.cpp:742<br>
#8 0x10ccab04a in clang::Preprocessor::HandleIncludeDirective(clang::SourceLocation, clang::Token&, clang::DirectoryLookup const*, clang::FileEntry const*, bool) PPDirectives.cpp:1709<br>
#9 0x10cca603a in clang::Preprocessor::HandleDirective(clang::Token&) PPDirectives.cpp:913<br>
#10 0x10cc2b042 in clang::Lexer::LexTokenInternal(clang::Token&, bool) Lexer.cpp:3671<br>
#11 0x10cd34636 in clang::Preprocessor::Lex(clang::Token&) Preprocessor.cpp:728<br>
#12 0x10ab0567a in clang::Parser::ConsumeToken() Parser.h:295<br>
#13 0x10a9bed0d in clang::ParseAST(clang::Sema&, bool, bool) ParseAST.cpp:139<br>
#14 0x10990fbca in clang::FrontendAction::Execute() FrontendAction.cpp:457<br>
#15 0x10985a498 in clang::CompilerInstance::ExecuteAction(clang::FrontendAction&) CompilerInstance.cpp:876<br>
#16 0x1099c1e73 in clang::ExecuteCompilerInvocation(clang::CompilerInstance*) ExecuteCompilerInvocation.cpp:241<br>
#17 0x105773f3f in cc1_main(llvm::ArrayRef<char const*>, char const*, void*) cc1_main.cpp:116<br>
#18 0x10576b806 in main driver.cpp:301<br>
#19 0x7fff9b1545ac in start (libdyld.dylib+0x35ac)<br>
#20 0xb (<unknown module>)<br>
<br>
0x60700000a580 is located 0 bytes inside of 80-byte region [0x60700000a580,0x60700000a5d0)<br>
freed by thread T0 here:<br>
#0 0x116699e0b in wrap__ZdlPv asan_new_delete.cc:121<br>
#1 0x108ed6759 in clang::vfs::File::getName() VirtualFileSystem.h:33<br>
#2 0x108dc3fba in clang::FileManager::getFile(llvm::StringRef, bool, bool) FileManager.cpp:316<br>
#3 0x10cbf5d53 in clang::HeaderSearch::getFileAndSuggestModule(llvm::StringRef, clang::SourceLocation, clang::DirectoryEntry const*, bool, clang::Module*, clang::ModuleMap::KnownHeader*) HeaderSearch.cpp:258<br>
#4 0x10cbf64d5 in clang::DirectoryLookup::LookupFile(llvm::StringRef&, clang::HeaderSearch&, clang::SourceLocation, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::Module*, clang::ModuleMap::KnownHeader*, bool&, bool&, llvm::SmallVectorImpl<char>&) const HeaderSearch.cpp:302<br>
#5 0x10cbf96d2 in clang::HeaderSearch::LookupFile(llvm::StringRef, clang::SourceLocation, bool, clang::DirectoryLookup const*, clang::DirectoryLookup const*&, llvm::ArrayRef<std::__1::pair<clang::FileEntry const*, clang::DirectoryEntry const*> >, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::Module*, clang::ModuleMap::KnownHeader*, bool, bool) HeaderSearch.cpp:717<br>
#6 0x10cca475e in clang::Preprocessor::LookupFile(clang::SourceLocation, llvm::StringRef, bool, clang::DirectoryLookup const*, clang::FileEntry const*, clang::DirectoryLookup const*&, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::ModuleMap::KnownHeader*, bool) PPDirectives.cpp:742<br>
#7 0x10ccab04a in clang::Preprocessor::HandleIncludeDirective(clang::SourceLocation, clang::Token&, clang::DirectoryLookup const*, clang::FileEntry const*, bool) PPDirectives.cpp:1709<br>
#8 0x10cca603a in clang::Preprocessor::HandleDirective(clang::Token&) PPDirectives.cpp:913<br>
#9 0x10cc2b042 in clang::Lexer::LexTokenInternal(clang::Token&, bool) Lexer.cpp:3671<br>
#10 0x10cd34636 in clang::Preprocessor::Lex(clang::Token&) Preprocessor.cpp:728<br>
#11 0x10ab0567a in clang::Parser::ConsumeToken() Parser.h:295<br>
#12 0x10a9bed0d in clang::ParseAST(clang::Sema&, bool, bool) ParseAST.cpp:139<br>
#13 0x10990fbca in clang::FrontendAction::Execute() FrontendAction.cpp:457<br>
#14 0x10985a498 in clang::CompilerInstance::ExecuteAction(clang::FrontendAction&) CompilerInstance.cpp:876<br>
#15 0x1099c1e73 in clang::ExecuteCompilerInvocation(clang::CompilerInstance*) ExecuteCompilerInvocation.cpp:241<br>
#16 0x105773f3f in cc1_main(llvm::ArrayRef<char const*>, char const*, void*) cc1_main.cpp:116<br>
#17 0x10576b806 in main driver.cpp:301<br>
#18 0x7fff9b1545ac in start (libdyld.dylib+0x35ac)<br>
#19 0xb (<unknown module>)<br>
<br>
previously allocated by thread T0 here:<br>
#0 0x11669984b in wrap__Znwm asan_new_delete.cc:74<br>
#1 0x7fff8ad36204 in std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >::__init(char const*, unsigned long) (libc++.1.dylib+0x3f204)<br>
#2 0x108ed92f9 in llvm::ErrorOr<clang::vfs::Status>::ErrorOr<clang::vfs::Status&>(clang::vfs::Status&&&, std::__1::enable_if<std::is_convertible<clang::vfs::Status&, clang::vfs::Status>::value, void>::type*) VirtualFileSystem.h:33<br>
#3 0x108ee17a7 in (anonymous namespace)::FileWithFixedStatus::status() ErrorOr.h:100<br>
#4 0x108ed65ee in clang::vfs::File::getName() VirtualFileSystem.h:96<br>
#5 0x108dc3fba in clang::FileManager::getFile(llvm::StringRef, bool, bool) FileManager.cpp:316<br>
#6 0x10cbf5d53 in clang::HeaderSearch::getFileAndSuggestModule(llvm::StringRef, clang::SourceLocation, clang::DirectoryEntry const*, bool, clang::Module*, clang::ModuleMap::KnownHeader*) HeaderSearch.cpp:258<br>
#7 0x10cbf64d5 in clang::DirectoryLookup::LookupFile(llvm::StringRef&, clang::HeaderSearch&, clang::SourceLocation, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::Module*, clang::ModuleMap::KnownHeader*, bool&, bool&, llvm::SmallVectorImpl<char>&) const HeaderSearch.cpp:302<br>
#8 0x10cbf96d2 in clang::HeaderSearch::LookupFile(llvm::StringRef, clang::SourceLocation, bool, clang::DirectoryLookup const*, clang::DirectoryLookup const*&, llvm::ArrayRef<std::__1::pair<clang::FileEntry const*, clang::DirectoryEntry const*> >, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::Module*, clang::ModuleMap::KnownHeader*, bool, bool) HeaderSearch.cpp:717<br>
#9 0x10cca475e in clang::Preprocessor::LookupFile(clang::SourceLocation, llvm::StringRef, bool, clang::DirectoryLookup const*, clang::FileEntry const*, clang::DirectoryLookup const*&, llvm::SmallVectorImpl<char>*, llvm::SmallVectorImpl<char>*, clang::ModuleMap::KnownHeader*, bool) PPDirectives.cpp:742<br>
#10 0x10ccab04a in clang::Preprocessor::HandleIncludeDirective(clang::SourceLocation, clang::Token&, clang::DirectoryLookup const*, clang::FileEntry const*, bool) PPDirectives.cpp:1709<br>
#11 0x10cca603a in clang::Preprocessor::HandleDirective(clang::Token&) PPDirectives.cpp:913<br>
#12 0x10cc2b042 in clang::Lexer::LexTokenInternal(clang::Token&, bool) Lexer.cpp:3671<br>
#13 0x10cd34636 in clang::Preprocessor::Lex(clang::Token&) Preprocessor.cpp:728<br>
#14 0x10ab0567a in clang::Parser::ConsumeToken() Parser.h:295<br>
#15 0x10a9bed0d in clang::ParseAST(clang::Sema&, bool, bool) ParseAST.cpp:139<br>
#16 0x10990fbca in clang::FrontendAction::Execute() FrontendAction.cpp:457<br>
#17 0x10985a498 in clang::CompilerInstance::ExecuteAction(clang::FrontendAction&) CompilerInstance.cpp:876<br>
#18 0x1099c1e73 in clang::ExecuteCompilerInvocation(clang::CompilerInstance*) ExecuteCompilerInvocation.cpp:241<br>
#19 0x105773f3f in cc1_main(llvm::ArrayRef<char const*>, char const*, void*) cc1_main.cpp:116<br>
#20 0x10576b806 in main driver.cpp:301<br>
#21 0x7fff9b1545ac in start (libdyld.dylib+0x35ac)<br>
#22 0xb (<unknown module>)<br>
<br>
SUMMARY: AddressSanitizer: heap-use-after-free asan_interceptors.cc:445 in wrap_memmove<br>
Shadow bytes around the buggy address:<br>
0x1c0e00001460: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa<br>
0x1c0e00001470: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa<br>
0x1c0e00001480: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa<br>
0x1c0e00001490: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa<br>
0x1c0e000014a0: fa fa 00 00 00 00 00 00 00 00 00 00 fa fa fa fa<br>
=>0x1c0e000014b0:[fd]fd fd fd fd fd fd fd fd fd fa fa fa fa fd fd<br>
0x1c0e000014c0: fd fd fd fd fd fd fd fd fa fa fa fa 00 00 00 00<br>
0x1c0e000014d0: 00 00 00 00 00 00 fa fa fa fa fd fd fd fd fd fd<br>
0x1c0e000014e0: fd fd fd fd fa fa fa fa 00 00 00 00 00 00 00 00<br>
0x1c0e000014f0: 00 00 fa fa fa fa fd fd fd fd fd fd fd fd fd fd<br>
0x1c0e00001500: fa fa fa fa fd fd fd fd fd fd fd fd fd fd fa fa<br>
Shadow byte legend (one shadow byte represents 8 application bytes):<br>
Addressable: 00<br>
Partially addressable: 01 02 03 04 05 06 07<br>
Heap left redzone: fa<br>
Heap right redzone: fb<br>
Freed heap region: fd<br>
Stack left redzone: f1<br>
Stack mid redzone: f2<br>
Stack right redzone: f3<br>
Stack partial redzone: f4<br>
Stack after return: f5<br>
Stack use after scope: f8<br>
Global redzone: f9<br>
Global init order: f6<br>
Poisoned by user: f7<br>
Container overflow: fc<br>
Array cookie: ac<br>
Intra object redzone: bb<br>
ASan internal: fe<br>
Left alloca redzone: ca<br>
Right alloca redzone: cb<br>
==77391==ABORTING<br>
Stack dump:<br>
0. Program arguments: /Users/bogner/build/llvm-san/./bin/clang -cc1 -internal-isystem /Volumes/NewYork/bogner/build/llvm-san/bin/../lib/clang/3.9.0/include -nostdsysteminc -Werror -I . -ivfsoverlay /Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.tmp.yaml -fsyntax-only /Volumes/NewYork/bogner/code/llvm/tools/clang/test/VFS/relative-path.c<br>
1. /Volumes/NewYork/bogner/code/llvm/tools/clang/test/VFS/relative-path.c:7:2: current parser token 'include'<br>
/Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.script: line 4: 77391 Abort trap: 6 /Users/bogner/build/llvm-san/./bin/clang -cc1 -internal-isystem /Volumes/NewYork/bogner/build/llvm-san/bin/../lib/clang/3.9.0/include -nostdsysteminc -Werror -I . -ivfsoverlay /Volumes/NewYork/bogner/build/llvm-san/tools/clang/test/VFS/Output/relative-path.c.tmp.yaml -fsyntax-only /Volumes/NewYork/bogner/code/llvm/tools/clang/test/VFS/relative-path.c<br><br>
> return &UFE;<br>
> }<br>
><br>
><br>
> Modified: cfe/trunk/lib/Basic/VirtualFileSystem.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/VirtualFileSystem.cpp?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/VirtualFileSystem.cpp?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/lib/Basic/VirtualFileSystem.cpp (original)<br>
> +++ cfe/trunk/lib/Basic/VirtualFileSystem.cpp Fri Jun 3 13:52:51 2016<br>
> @@ -140,16 +140,19 @@ namespace {<br>
> class RealFile : public File {<br>
> int FD;<br>
> Status S;<br>
> + std::string RealName;<br>
> friend class RealFileSystem;<br>
> - RealFile(int FD, StringRef NewName)<br>
> + RealFile(int FD, StringRef NewName, StringRef NewRealPathName)<br>
> : FD(FD), S(NewName, {}, {}, {}, {}, {},<br>
> - llvm::sys::fs::file_type::status_error, {}) {<br>
> + llvm::sys::fs::file_type::status_error, {}),<br>
> + RealName(NewRealPathName.str()) {<br>
> assert(FD >= 0 && "Invalid or inactive file descriptor");<br>
> }<br>
><br>
> public:<br>
> ~RealFile() override;<br>
> ErrorOr<Status> status() override;<br>
> + ErrorOr<StringRef> getName() override;<br>
> ErrorOr<std::unique_ptr<MemoryBuffer>> getBuffer(const Twine &Name,<br>
> int64_t FileSize,<br>
> bool RequiresNullTerminator,<br>
> @@ -170,6 +173,10 @@ ErrorOr<Status> RealFile::status() {<br>
> return S;<br>
> }<br>
><br>
> +ErrorOr<StringRef> RealFile::getName() {<br>
> + return RealName.empty() ? S.getName() : StringRef(RealName);<br>
> +}<br>
> +<br>
> ErrorOr<std::unique_ptr<MemoryBuffer>><br>
> RealFile::getBuffer(const Twine &Name, int64_t FileSize,<br>
> bool RequiresNullTerminator, bool IsVolatile) {<br>
> @@ -207,9 +214,10 @@ ErrorOr<Status> RealFileSystem::status(c<br>
> ErrorOr<std::unique_ptr<File>><br>
> RealFileSystem::openFileForRead(const Twine &Name) {<br>
> int FD;<br>
> - if (std::error_code EC = sys::fs::openFileForRead(Name, FD))<br>
> + SmallString<256> RealName;<br>
> + if (std::error_code EC = sys::fs::openFileForRead(Name, FD, &RealName))<br>
> return EC;<br>
> - return std::unique_ptr<File>(new RealFile(FD, Name.str()));<br>
> + return std::unique_ptr<File>(new RealFile(FD, Name.str(), RealName.str()));<br>
> }<br>
><br>
> llvm::ErrorOr<std::string> RealFileSystem::getCurrentWorkingDirectory() const {<br>
><br>
> Modified: cfe/trunk/lib/Lex/HeaderSearch.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/HeaderSearch.cpp?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/HeaderSearch.cpp?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/lib/Lex/HeaderSearch.cpp (original)<br>
> +++ cfe/trunk/lib/Lex/HeaderSearch.cpp Fri Jun 3 13:52:51 2016<br>
> @@ -250,8 +250,9 @@ const char *DirectoryLookup::getName() c<br>
> }<br>
><br>
> const FileEntry *HeaderSearch::getFileAndSuggestModule(<br>
> - StringRef FileName, const DirectoryEntry *Dir, bool IsSystemHeaderDir,<br>
> - Module *RequestingModule, ModuleMap::KnownHeader *SuggestedModule) {<br>
> + StringRef FileName, SourceLocation IncludeLoc, const DirectoryEntry *Dir,<br>
> + bool IsSystemHeaderDir, Module *RequestingModule,<br>
> + ModuleMap::KnownHeader *SuggestedModule) {<br>
> // If we have a module map that might map this header, load it and<br>
> // check whether we'll have a suggestion for a module.<br>
> const FileEntry *File = getFileMgr().getFile(FileName, /*OpenFile=*/true);<br>
> @@ -272,6 +273,7 @@ const FileEntry *HeaderSearch::getFileAn<br>
> const FileEntry *DirectoryLookup::LookupFile(<br>
> StringRef &Filename,<br>
> HeaderSearch &HS,<br>
> + SourceLocation IncludeLoc,<br>
> SmallVectorImpl<char> *SearchPath,<br>
> SmallVectorImpl<char> *RelativePath,<br>
> Module *RequestingModule,<br>
> @@ -297,7 +299,7 @@ const FileEntry *DirectoryLookup::Lookup<br>
> RelativePath->append(Filename.begin(), Filename.end());<br>
> }<br>
><br>
> - return HS.getFileAndSuggestModule(TmpDir, getDir(),<br>
> + return HS.getFileAndSuggestModule(TmpDir, IncludeLoc, getDir(),<br>
> isSystemHeaderDirectory(),<br>
> RequestingModule, SuggestedModule);<br>
> }<br>
> @@ -585,7 +587,7 @@ const FileEntry *HeaderSearch::LookupFil<br>
> RelativePath->append(Filename.begin(), Filename.end());<br>
> }<br>
> // Otherwise, just return the file.<br>
> - return getFileAndSuggestModule(Filename, nullptr,<br>
> + return getFileAndSuggestModule(Filename, IncludeLoc, nullptr,<br>
> /*IsSystemHeaderDir*/false,<br>
> RequestingModule, SuggestedModule);<br>
> }<br>
> @@ -622,7 +624,7 @@ const FileEntry *HeaderSearch::LookupFil<br>
> Includer ? getFileInfo(Includer).DirInfo != SrcMgr::C_User :<br>
> BuildSystemModule;<br>
> if (const FileEntry *FE = getFileAndSuggestModule(<br>
> - TmpDir, IncluderAndDir.second, IncluderIsSystemHeader,<br>
> + TmpDir, IncludeLoc, IncluderAndDir.second, IncluderIsSystemHeader,<br>
> RequestingModule, SuggestedModule)) {<br>
> if (!Includer) {<br>
> assert(First && "only first includer can have no file");<br>
> @@ -713,7 +715,7 @@ const FileEntry *HeaderSearch::LookupFil<br>
> bool InUserSpecifiedSystemFramework = false;<br>
> bool HasBeenMapped = false;<br>
> const FileEntry *FE = SearchDirs[i].LookupFile(<br>
> - Filename, *this, SearchPath, RelativePath, RequestingModule,<br>
> + Filename, *this, IncludeLoc, SearchPath, RelativePath, RequestingModule,<br>
> SuggestedModule, InUserSpecifiedSystemFramework, HasBeenMapped,<br>
> MappedName);<br>
> if (HasBeenMapped) {<br>
><br>
> Modified: cfe/trunk/lib/Lex/PPDirectives.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPDirectives.cpp?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPDirectives.cpp?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/lib/Lex/PPDirectives.cpp (original)<br>
> +++ cfe/trunk/lib/Lex/PPDirectives.cpp Fri Jun 3 13:52:51 2016<br>
> @@ -24,6 +24,9 @@<br>
> #include "clang/Lex/ModuleLoader.h"<br>
> #include "clang/Lex/Pragma.h"<br>
> #include "llvm/ADT/APInt.h"<br>
> +#include "llvm/ADT/STLExtras.h"<br>
> +#include "llvm/ADT/StringExtras.h"<br>
> +#include "llvm/ADT/iterator_range.h"<br>
> #include "llvm/Support/ErrorHandling.h"<br>
> #include "llvm/Support/Path.h"<br>
> #include "llvm/Support/SaveAndRestore.h"<br>
> @@ -1556,6 +1559,41 @@ static void diagnoseAutoModuleImport(<br>
> ("@import " + PathString + ";").str());<br>
> }<br>
><br>
> +namespace {<br>
> + // Given a vector of path components and a string containing the real<br>
> + // path to the file, build a properly-cased replacement in the vector,<br>
> + // and return true if the replacement should be suggested.<br>
> + bool TrySimplifyPath(SmallVectorImpl<StringRef> &Components,<br>
> + StringRef RealPathName) {<br>
> + auto RealPathComponentIter = llvm::sys::path::rbegin(RealPathName);<br>
> + auto RealPathComponentEnd = llvm::sys::path::rend(RealPathName);<br>
> + int Cnt = 0;<br>
> + bool SuggestReplacement = false;<br>
> + // Below is a best-effort to handle ".." in paths. It is admittedly<br>
> + // not 100% correct in the presence of symlinks.<br>
> + for(auto &Component : llvm::reverse(Components)) {<br>
> + if ("." == Component) {<br>
> + } else if (".." == Component) {<br>
> + ++Cnt;<br>
> + } else if (Cnt) {<br>
> + --Cnt;<br>
> + } else if (RealPathComponentIter != RealPathComponentEnd) {<br>
> + if (Component != *RealPathComponentIter) {<br>
> + // If these path components differ by more than just case, then we<br>
> + // may be looking at symlinked paths. Bail on this diagnostic to avoid<br>
> + // noisy false positives.<br>
> + SuggestReplacement = RealPathComponentIter->equals_lower(Component);<br>
> + if (!SuggestReplacement)<br>
> + break;<br>
> + Component = *RealPathComponentIter;<br>
> + }<br>
> + ++RealPathComponentIter;<br>
> + }<br>
> + }<br>
> + return SuggestReplacement;<br>
> + }<br>
> +}<br>
> +<br>
> /// HandleIncludeDirective - The "\#include" tokens have just been read, read<br>
> /// the file to be included from the lexer, then include it! This is a common<br>
> /// routine with functionality shared between \#include, \#include_next and<br>
> @@ -1720,6 +1758,35 @@ void Preprocessor::HandleIncludeDirectiv<br>
> }<br>
> }<br>
><br>
> + // Issue a diagnostic if the name of the file on disk has a different case<br>
> + // than the one we're about to open.<br>
> + const bool CheckIncludePathPortability =<br>
> + File && !File->tryGetRealPathName().empty();<br>
> +<br>
> + if (CheckIncludePathPortability) {<br>
> + StringRef Name = LangOpts.MSVCCompat ? NormalizedPath.c_str() : Filename;<br>
> + StringRef RealPathName = File->tryGetRealPathName();<br>
> + SmallVector<StringRef, 16> Components(llvm::sys::path::begin(Name),<br>
> + llvm::sys::path::end(Name));<br>
> +<br>
> + if (TrySimplifyPath(Components, RealPathName)) {<br>
> + SmallString<128> Path;<br>
> + Path.reserve(Name.size()+2);<br>
> + Path.push_back(isAngled ? '<' : '"');<br>
> + for (auto Component : Components) {<br>
> + Path.append(Component);<br>
> + // Append the separator the user used, or the close quote<br>
> + Path.push_back(<br>
> + Path.size() <= Filename.size() ? Filename[Path.size()-1] :<br>
> + (isAngled ? '>' : '"'));<br>
> + }<br>
> + auto Replacement = Path.str().str();<br>
> + SourceRange Range(FilenameTok.getLocation(), CharEnd);<br>
> + Diag(FilenameTok, diag::pp_nonportable_path) << Replacement <<<br>
> + FixItHint::CreateReplacement(Range, Replacement);<br>
> + }<br>
> + }<br>
> +<br>
> // Should we enter the source file? Set to false if either the source file is<br>
> // known to have no effect beyond its effect on module visibility -- that is,<br>
> // if it's got an include guard that is already defined or is a modular header<br>
><br>
> Added: cfe/trunk/test/Lexer/Inputs/case-insensitive-include.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Lexer/Inputs/case-insensitive-include.h?rev=271708&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Lexer/Inputs/case-insensitive-include.h?rev=271708&view=auto</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/Lexer/Inputs/case-insensitive-include.h (added)<br>
> +++ cfe/trunk/test/Lexer/Inputs/case-insensitive-include.h Fri Jun 3 13:52:51 2016<br>
> @@ -0,0 +1,5 @@<br>
> +#pragma once<br>
> +<br>
> +struct S {<br>
> + int x;<br>
> +};<br>
><br>
> Added: cfe/trunk/test/Lexer/case-insensitive-include-ms.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Lexer/case-insensitive-include-ms.c?rev=271708&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Lexer/case-insensitive-include-ms.c?rev=271708&view=auto</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/Lexer/case-insensitive-include-ms.c (added)<br>
> +++ cfe/trunk/test/Lexer/case-insensitive-include-ms.c Fri Jun 3 13:52:51 2016<br>
> @@ -0,0 +1,18 @@<br>
> +// REQUIRES: case-insensitive-filesystem<br>
> +<br>
> +// RUN: mkdir -p %T/apath<br>
> +// RUN: cp %S/Inputs/case-insensitive-include.h %T<br>
> +// RUN: cd %T<br>
> +// RUN: %clang_cc1 -fsyntax-only -fms-compatibility %s -include %s -I %T -verify<br>
> +// RUN: %clang_cc1 -fsyntax-only -fms-compatibility -fdiagnostics-parseable-fixits %s -include %s -I %T 2>&1 | FileCheck %s<br>
> +<br>
> +#include "..\Output\.\case-insensitive-include.h"<br>
> +#include "..\Output\.\Case-Insensitive-Include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:50}:"\"..\\Output\\.\\case-insensitive-include.h\""<br>
> +#include "..\output\.\case-insensitive-include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:50}:"\"..\\Output\\.\\case-insensitive-include.h\""<br>
> +<br>
> +#include "apath\..\.\case-insensitive-include.h"<br>
> +#include "apath\..\.\Case-Insensitive-Include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:49}:"\"apath\\..\\.\\case-insensitive-include.h\""<br>
> +#include "APath\..\.\case-insensitive-include.h" // For the sake of efficiency, this case is not diagnosed. :-(<br>
><br>
> Added: cfe/trunk/test/Lexer/case-insensitive-include.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Lexer/case-insensitive-include.c?rev=271708&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Lexer/case-insensitive-include.c?rev=271708&view=auto</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/Lexer/case-insensitive-include.c (added)<br>
> +++ cfe/trunk/test/Lexer/case-insensitive-include.c Fri Jun 3 13:52:51 2016<br>
> @@ -0,0 +1,27 @@<br>
> +// REQUIRES: case-insensitive-filesystem<br>
> +<br>
> +// RUN: mkdir -p %T/apath<br>
> +// RUN: cp %S/Inputs/case-insensitive-include.h %T<br>
> +// RUN: cd %T<br>
> +// RUN: %clang_cc1 -fsyntax-only %s -include %s -I %T -verify<br>
> +// RUN: %clang_cc1 -fsyntax-only -fdiagnostics-parseable-fixits %s -include %s -I %T 2>&1 | FileCheck %s<br>
> +<br>
> +#include "case-insensitive-include.h"<br>
> +#include "Case-Insensitive-Include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:38}:"\"case-insensitive-include.h\""<br>
> +<br>
> +#include "../Output/./case-insensitive-include.h"<br>
> +#include "../Output/./Case-Insensitive-Include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:50}:"\"../Output/./case-insensitive-include.h\""<br>
> +#include "../output/./case-insensitive-include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:50}:"\"../Output/./case-insensitive-include.h\""<br>
> +<br>
> +#include "apath/.././case-insensitive-include.h"<br>
> +#include "apath/.././Case-Insensitive-Include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:49}:"\"apath/.././case-insensitive-include.h\""<br>
> +#include "APath/.././case-insensitive-include.h" // For the sake of efficiency, this case is not diagnosed. :-(<br>
> +<br>
> +#include "../Output/./apath/.././case-insensitive-include.h"<br>
> +#include "../Output/./APath/.././case-insensitive-include.h" // For the sake of efficiency, this case is not diagnosed. :-(<br>
> +#include "../output/./apath/.././case-insensitive-include.h" // expected-warning {{non-portable path}}<br>
> +// CHECK: fix-it:"{{.*}}":{[[@LINE-1]]:10-[[@LINE-1]]:61}:"\"../Output/./apath/.././case-insensitive-include.h\""<br>
><br>
> Modified: cfe/trunk/test/PCH/case-insensitive-include.c<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/PCH/case-insensitive-include.c?rev=271708&r1=271707&r2=271708&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/PCH/case-insensitive-include.c?rev=271708&r1=271707&r2=271708&view=diff</a><br>
> ==============================================================================<br>
> --- cfe/trunk/test/PCH/case-insensitive-include.c (original)<br>
> +++ cfe/trunk/test/PCH/case-insensitive-include.c Fri Jun 3 13:52:51 2016<br>
> @@ -2,7 +2,7 @@<br>
><br>
> // Test this without pch.<br>
> // RUN: cp %S/Inputs/case-insensitive-include.h %T<br>
> -// RUN: %clang_cc1 -fsyntax-only %s -include %s -I %T -verify<br>
> +// RUN: %clang_cc1 -Wno-nonportable-include-path -fsyntax-only %s -include %s -I %T -verify<br>
><br>
> // Test with pch.<br>
> // RUN: %clang_cc1 -emit-pch -o %t.pch %s -I %T<br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Bruno Cardoso Lopes <br><a href="http://www.brunocardoso.cc" target="_blank">http://www.brunocardoso.cc</a></div>
</div>