gcc icon indicating copy to clipboard operation
gcc copied to clipboard

13185 -zassert-deflib does not work for 64-bit objects (9.3.0)

Open citrus-it opened this issue 4 years ago • 3 comments

https://www.illumos.org/issues/13185

Using the test case from the above issue, with the fix applied:

+ gcc 10, 32
ld: warning: dynamic library found on default search path (/usr/lib): libmtmalloc.so
collect2: error: ld returned 1 exit status

+ gcc 10, 64
ld: warning: dynamic library found on default search path (/usr/lib/amd64): libmtmalloc.so
collect2: error: ld returned 1 exit status

citrus-it avatar Jan 11 '21 14:01 citrus-it

This seems entirely reasonable. Did you run the any of the tests that this may affect?

richlowe avatar Jan 11 '21 19:01 richlowe

This seems entirely reasonable. Did you run the any of the tests that this may affect?

Not yet, but I have run a full gate build with a patched gcc7 and run wsdiff (which was clean apart from the usual suspects)

citrus-it avatar Jan 11 '21 19:01 citrus-it

I've run the testsuite against this gcc7 branch (with both pending PRs applied):

                === gcc Summary for unix//-m32 ===
# of expected passes            118469
# of unexpected failures        116
# of unexpected successes       1
# of expected failures          371
# of unresolved testcases       53
# of unsupported tests          2134
                === gcc Summary for unix//-m64 ===
# of expected passes            120061
# of unexpected failures        134
# of unexpected successes       1
# of expected failures          365
# of unresolved testcases       52
# of unsupported tests          2065
                === gcc Summary for unix//-m64/-msave-args ===
# of expected passes            120056
# of unexpected failures        132
# of unexpected successes       1
# of expected failures          365
# of unresolved testcases       52
# of unsupported tests          2069
                === gcc Summary ===
# of expected passes            358586
# of unexpected failures        382
# of unexpected successes       3
# of expected failures          1101
# of unresolved testcases       157
# of unsupported tests          6268

I'm going to run it again without these two patches applied for comparison.

citrus-it avatar Jan 14 '21 18:01 citrus-it