Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
wants to merge 1 commit into
base: il-9_3_0
Choose a base branch
from

Conversation

citrus-it
Copy link
Member

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

@richlowe
Copy link
Member

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

@citrus-it
Copy link
Member Author

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 citrus-it changed the title 13185 -zassert-deflib does not work for 64-bit objects 13185 -zassert-deflib does not work for 64-bit objects (9.3.0) Jan 12, 2021
@citrus-it
Copy link
Member Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants