Remove TypeNodes.def from the modulemap.
authorJohn McCall <rjmccall@apple.com>
Wed, 2 Oct 2019 01:02:27 +0000 (01:02 +0000)
committerJohn McCall <rjmccall@apple.com>
Wed, 2 Oct 2019 01:02:27 +0000 (01:02 +0000)
We currently just look for files named in the modulemap in its
associated source directory.  This means that we can't name
generated files, like TypeNodes.def now is, which means we can't
explicitly mark it as textual.  But fortunately that's okay
because (as I understand it) the most important purpose of naming
the header in the modulemap is to ensure that it's not treated as
public, and the search for public headers also only considers
files in the associated source directory.  This isn't an elegant
solution, since among other things it means that a build which
wrote the generated files directly into the source directory would
result in something that wouldn't build as a module, but that's
a problem for all our other generated files as well.

llvm-svn: 373416

clang/include/clang/module.modulemap

index fc4a2de..f7bf482 100644 (file)
@@ -20,7 +20,6 @@ module Clang_AST {
   textual header "AST/BuiltinTypes.def"
   textual header "AST/OperationKinds.def"
   textual header "AST/TypeLocNodes.def"
-  textual header "AST/TypeNodes.def"
 
   module * { export * }
 }