[m-rev.] for review: simplify the creation of .int3 files

Julien Fischer jfischer at opturion.com
Thu Jan 31 13:07:19 AEDT 2019


Hi Zoltan,

On Tue, 29 Jan 2019, Zoltan Somogyi wrote:

> The diff is for review by Julien or Peter, but I would like everyone's
> feedback on the plans above.

I have no objections to your plan -- that said, it's been a while
since I worked on that part of the compiler so I've probably forgotten
quite a few things about it.

> To make a review easier, I am attaching the summary of how the pre-diff
> compiler computed what goes into a .int3 file. I would suggest that
> whoever does the review, do it in two stages: check whether the summary
> is a correct description of what the old code does,

The description looks correct.

> and then check whether
> what the new code does for each kind of item is what the summary calls for
> that kind of item. Otherwise, the correspondence between the too-complex
> old code and the simple new code is too hard to keep track of in one's head.
>
> I am also attaching the output of git diff both with and without -b.

As you wish to continue working on it, I'm happy for you to commit now.  I had
a look through you the latest version of your change and couldn't see anything
obviously incorrect.

Julien.


More information about the reviews mailing list