en Copyright 2015 Java http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap#8839e278 [Modules] Improve error message when cannot find parent module for submodule definition.Before the change the diagnostic for module unknown.submodule {}was "error: expected module name" which is incorrect and misleadingbecause both "unknown" and "submodule" are valid module names.We already have a better error message when a parent module is asubmodule itself and is missing. Make the error for a missing top-levelmodule more like the one for a submodule.rdar://problem/64424407Reviewed By: brunoDifferential Revision: https://reviews.llvm.org/D84458 List of files: /llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap Thu, 23 Jul 2020 19:47:16 +0000 Volodymyr Sapsai <vsapsai@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap#49532137 Make AST reading work better with LLVM_APPEND_VC_REV=NOWith LLVM_APPEND_VC_REV=NO, Modules/merge-lifetime-extended-temporary.cppwould fail if it ran before a0f50d731639350c7a7 (which changedthe serialization format) and then after, for these reasons:1. With LLVM_APPEND_VC_REV=NO, the module hash before and after the change was the same.2. Modules/merge-lifetime-extended-temporary.cpp is the only test we have that uses -fmodule-cache-path=%t that a) actually writes to the cache path b) doesn't do `rm -rf %t` at the top of the testSo the old run would write a module file, and then the new run wouldtry to load it, but the serialized format changed.Do several things to fix this:1. Include clang::serialization::VERSION_MAJOR/VERSION_MINOR in the module hash, so that when the AST format changes (...and we remember to bump these), we use a different module cache dir.2. Bump VERSION_MAJOR, since a0f50d731639350c7a7 changed the on-disk format in a way that a gch file written before that change can't be read after that change.3. Add `rm -rf %t` to all tests that pass -fmodule-cache-path=%t. This is unnecessary from a correctness PoV after 1 and 2, but makes it so that we don't amass many cache dirs over time. (Arguably, it also makes it so that the test suite doesn't catch when we change the serialization format but don't bump clang::serialization::VERSION_MAJOR/VERSION_MINOR; oh well.)Differential Revision: https://reviews.llvm.org/D73202 List of files: /llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap Wed, 22 Jan 2020 15:34:34 +0000 Nico Weber <thakis@chromium.org> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap#040e1266 Support lazy stat'ing of files referenced by module maps.This patch adds support for a `header` declaration in a module map to specifycertain `stat` information (currently, size and mtime) about that header file.This has two purposes:- It removes the need to eagerly `stat` every file referenced by a module map. Instead, we track a list of unresolved header files with each size / mtime (actually, for simplicity, we track submodules with such headers), and when attempting to look up a header file based on a `FileEntry`, we check if there are any unresolved header directives with that `FileEntry`'s size / mtime and perform deferred `stat`s if so.- It permits a preprocessed module to be compiled without the original files being present on disk. The only reason we used to need those files was to get the `stat` information in order to do header -> module lookups when using the module. If we're provided with the `stat` information in the preprocessed module, we can avoid requiring the files to exist.Unlike most `header` directives, if a `header` directive with `stat`information has no corresponding on-disk file the enclosing module is *not*marked unavailable (so that behavior is consistent regardless of whether we'veresolved a header directive, and so that preprocessed modules don't get markedunavailable). We could actually do this for all `header` directives: the onlyreason we mark the module unavailable if headers are missing is to give adiagnostic slightly earlier (rather than waiting until we actually try to buildthe module / load and validate its .pcm file).Differential Revision: https://reviews.llvm.org/D33703llvm-svn: 304515 List of files: /llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap Fri, 02 Jun 2017 01:55:39 +0000 Richard Smith <richard-llvm@metafoo.co.uk> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap#ae6df27e [modules] When diagnosing errors in module map files found by 'extern module' declarations, show how we got to that module map file.llvm-svn: 242105 List of files: /llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap Tue, 14 Jul 2015 02:06:01 +0000 Richard Smith <richard-llvm@metafoo.co.uk> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap#8f4d3ff1 [modules] Restrict the module use-declaration to only appear in top-levelmodules, and allow sub-modules of a module with a use-declaration to make useof the nominated modules.llvm-svn: 233323 List of files: /llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap Thu, 26 Mar 2015 22:10:01 +0000 Richard Smith <richard-llvm@metafoo.co.uk> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap#0414b857 [modules] Accept //-style comments in module maps on purpose rather than byaccident, and accept them even when they begin '//*'.llvm-svn: 229240 List of files: /llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap Sat, 14 Feb 2015 05:32:00 +0000 Richard Smith <richard-llvm@metafoo.co.uk> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap#ac425e91 PR22299: Relocate code for handling -fmodule-map-file= so that we don't try toproduce diagnostics with source locations before the diagnostics system isready for them.llvm-svn: 226882 List of files: /llvm-project-15.0.7/clang/test/Modules/diagnostics.modulemap Fri, 23 Jan 2015 00:01:13 +0000 Richard Smith <richard-llvm@metafoo.co.uk>