Sorry, Legacy Header Units

Referring specifically to that part of the modules proposal

When a #include appears within non-modular code, if the named header file is known to correspond to a legacy header unit, the implementation treats the #include as an import of the corresponding legacy header unit. The mechanism for discovering this correspondence is left implementation-defined; there are multiple viable strategies here (such as explicitly building legacy header modules and providing them as input to downstream compilations, or introducing accompanying files describing the legacy header structure) and we wish to encourage exploration of this space. An implementation is also permitted to not provide any mapping mechanism, and process each legacy header unit independently


The list of such headers can only be maintained manually or synthesized by scanning the complete dependency tree - rather than be extractable from an arbitrary individual source file.




On Fri, 8 Feb 2019 at 18:20 Gabriel Dos Reis <gdr@microsoft.com> wrote:

I cannot understand what you are saying because I don’t know what LHU is, and I don’t understand the second bullet and why that is the case.

 

From: Corentin <corentin.jabot@gmail.com>
Sent: Friday, February 8, 2019 9:18 AM
To: Evolution Working Group mailing list <ext@lists.isocpp.org>
Cc: ben.boeckel@kitware.com; WG21 Tooling Study Group SG15 <tooling@open-std.org>; Gabriel Dos Reis <gdr@microsoft.com>
Subject: Re: [Ext] [Tooling] [D1483] How CMake supports Fortran modules and its applicability to C++

 

 

Nice document.

 

6.4  I believe LHU imported through include will need to be identified by either

  • Having a manually maintained list of LHU
  • Parsing all files in the projects/dependency and assuming that files that are not imported with import at least once are not LHU

 

I agree that letting the compiler decides to treat an include as an LHU without the blessing of the build system would be asking for trouble

 

 

On Fri, 8 Feb 2019 at 17:51 Gabriel Dos Reis via Ext <ext@lists.isocpp.org> wrote:

Early feedback regarding section 6.3 and 6.4.

Concern in 6.3:
The description there is notional, not a requirement that needs to be followed by the letter by compilers.
There is an alternative formulation that uses the notion or "semantics abstract graph" (term defined in the Modules TS) that does not rely on synthesized header unit.

Concern in 6.4:
The way to import of "header import" is that of using a precompiled header file.  If CMAKE already supports uses of PCHs, then the machinery is already there.
For every import of a header H, there would be a rule for building say H.pch unless there is already a H.pch supplied by the system or other sources.


Early feedback on section 7.
That ask in 7.1 looks immensely reasonable to me; we have been considering similar for a while.
The Visual C++ team would be happy to team up with other tool vendor to provide an open source version.

-- Gaby

| -----Original Message-----
| From: tooling-bounces@open-std.org <tooling-bounces@open-std.org> On
| Behalf Of Ben Boeckel
| Sent: Friday, February 8, 2019 7:55 AM
| To: WG21 Tooling Study Group SG15 <tooling@open-std.org>
| Subject: [Tooling] [D1483] How CMake supports Fortran modules and its
| applicability to C++
|
| Hi,
|
| Here is copy of Kitware's paper to be discussed at Kona. I have a PDF,
| but it was too large to attach to the list. I'll be at Kona, but the
| other authors are not able to make it.
|
| An HTML version is hosted here:
|
|
| https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmath
| stuf.fedorapeople.org%2Ffortran-modules%2Ffortran-
| modules.html&amp;data=02%7C01%7Cgdr%40microsoft.com%7Cd49f0fb63
| aac4ce886ff08d68dddd8b0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7
| C1%7C636852381303060160&amp;sdata=ltkTtPf84tUk76Am5swiz6eNBGqZW
| ydRfDoswsYEmlA%3D&amp;reserved=0
|
| Feedback welcome.
|
| Thanks,
|
| --Ben
| _______________________________________________
| Tooling mailing list
| Tooling@isocpp.open-std.org
| https://nam06.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.
| open-
| std.org%2Fmailman%2Flistinfo%2Ftooling&amp;data=02%7C01%7Cgdr%40m
| icrosoft.com%7Cd49f0fb63aac4ce886ff08d68dddd8b0%7C72f988bf86f141af9
| 1ab2d7cd011db47%7C1%7C1%7C636852381303060160&amp;sdata=YLene6t
| qdjuV7ad%2BAHf5kceett%2F1Whqj7Db0zka470g%3D&amp;reserved=0
_______________________________________________
Ext mailing list
Ext@lists.isocpp.org
Subscription: http://lists.isocpp.org/mailman/listinfo.cgi/ext
Link to this post: http://lists.isocpp.org/ext/2019/02/7507.php