[m-rev.] for review: start using --generate-dependencies-ints

Julien Fischer jfischer at opturion.com
Sat Oct 21 15:20:58 AEDT 2023


On Fri, 20 Oct 2023, Zoltan Somogyi wrote:

> On 2023-10-20 22:27 +11:00 AEDT, "Julien Fischer" <jfischer at opturion.com> wrote:
>> mmc --make (which is used to build the C# library) supports a no-op
>> depend target, but doesn't recongise depend_ints. So either need
>> similarly support depend_ints as a no-op target for mmc --make or
>> make the above target in the Mmakefile dependent on the grade and not
>> use depend_ints in the non-C grades.
>
> I think the former is preferable. Can you test whether just extending the filter
> on line 108 of make.top_level.m to cover ".depend_ints" would work?

I have committed a diff that does this and fixes the problem. There's
another issue with this change: it is orders of magnitude slower on
Windows.

On my Linux machine a top-level mmake depend is now ~17 seconds; on
Windows (using MSYS2) it is ~21 minutes. The primary culprit for this
seems to be all the file copying that is going on. While I suspect
the times are, more or less, equivalent to what was required for
building all the inteface files under the old approach, the main
problem is that there very little visual feedback on what is happening.
(I initially though that the compiler had gone into a loop, until I
turned on --verbose.)

Julien.


More information about the reviews mailing list