emxomf warning: Cycle detected by make_type.

[dmik]

When compiling large source files (like this: ​https://github.com/bitwiseworks/mozilla-os2/blob/master/content/base/src/nsDocument.cpp), you will often see this:

emxomf warning: Cycle detected by make_type.
emxomf info: parsing sym no.81366 type=64 at char ' ' in position 12:
slots:r54927

There is a patch to disable this warning in #220 (​emxomf-04-no-hll-warnings.diff​) but Knut claims that emxomf should be fixed instead as it produces incorrect HLL info.

This ticket is to sort it out.

[dmik]

Note that the current RPM builds have the emxomf-04-no-hll-warnings.diff applied for the time being.

[Yuri Dario]

Show warnings only in verbose mode.

[Yuri Dario]

This newer patch includes the previous one.

[dmik]

BTW, there are more warnings that happen to be produced when compiling large files (and not only them):

emxomf warning: Unknown range type: 16..0
emxomf info: parsing sym no.3 type=128 at char ' ' in position 22:
__float128:t1=r0;16;0;
emxomf warning: syntax error in stabs:   (off 22)
 stabs: __float128:t1=r0;16;0;
emxomf info: parsing sym no.3 type=128 at char ' ' in position 22:
__float128:t1=r0;16;0;

A good test case for all that is e.g. this file: ​https://github.com/bitwiseworks/mozilla-os2/blob/master/content/base/src/nsDocument.cpp. Note that this file was also the reason #293 was introduced by Steven. It would simply fail to compile with Index too large before and with the Steven's patch applied it gives just a warning:

emxomf warning: HLL debug suppressed for symbol '__ZN7mozilla15FullscreenRoots9sInstanceE' - index 33100 too large
emxomf warning: HLL debug suppressed for symbol '_gPendingPointerLockRequest' - index 39164 too large

I don't know though if it's something that may be fixed or it's an invincible OMF format limitation.

And to clarify, this ticket is meant to fix the source of the problem, not hide warnings — these are only the temporary solution, to stop annoying regular developers for which the emxomf warnings like the one in the subject are completely meaningless.

[dmik]

I'm uploading the current cumulative patch for verbose warnings that also includes the one from #251 and matches the RPM build of libc (emxomf) 0.6.5-20.

[bird]

r3920 implements suppressing stabs -> hll conversion warnings. By default there will be one warning that there are problems, but the user is spared all the details. For users which doesn't want that warning (or the one from #293), use set EMXOMFOPT=-q or specify -q on the emxomf command line. (Equally, for users wishing to see it all, adding the -v to the emxomf command line or set EMXOMFOPT=-v will show them like before.)

At present, I'm not going to spend time fixing the conversion issues that newer gcc versions present to emxomf, so I'm removing this ticked from the 0.6.6 milestone.