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.

Sign up for GitHub

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

Jump to bottom

CRASH: multiply 100 100 31416 #4

Closed
ascherer opened this issue Nov 26, 2018 · 5 comments
Closed

CRASH: multiply 100 100 31416 #4

ascherer opened this issue Nov 26, 2018 · 5 comments
Labels
bug

Comments

@ascherer
Copy link
Owner

The local version dies.
@ascherer ascherer added the bug label Nov 26, 2018
@ascherer
Copy link
Owner Author

ascherer commented Nov 26, 2018 •
edited
Loading

It seems to boil down to --without debuginfo. Optimization kills multiply; all other programs run normally and produce the expected results.

@ascherer
Copy link
Owner Author

This occurs even on master, if you replace -g with -O in Makefile.

ascherer added a commit that referenced this issue Nov 27, 2018
@ascherer
Fix issue #4 with temporary patch.
a835b56
@ascherer
Copy link
Owner Author

Fixed and reported upstream.

ascherer added a commit that referenced this issue Nov 27, 2018
@ascherer
Fix issue #4 with temporary patch.
8427af2
ascherer added a commit that referenced this issue Nov 27, 2018
@ascherer
Fix issue #4 with temporary patch.
c3b6d82
ascherer added a commit that referenced this issue Nov 30, 2018
@ascherer
Fix issue #4 with temporary patch.
7dd7699
ascherer added a commit that referenced this issue Dec 1, 2018
@ascherer
Fix issue #4 with temporary patch.
fa28f59
@ascherer
Copy link
Owner Author

ascherer commented Apr 11, 2019 •
edited
Loading

While gcc/g++ on Linux requires -Optimization to reveal the fatal memory error, Apple LLVM version 9.0.0 (clang-900.0.39.2) on macOS 10.12.6 produces a flawed multiply executable right out of the box. The following recipe with the original SGB tarball

$> tar xzvf sgb.tar.gz
$> make tests multiply
$> ./multiply 100 100 31416

reports Abort trap: 6.

Despite a detailed bug report, DEK hasn't come around to fixing this mistake upstream.

ascherer added a commit that referenced this issue Feb 1, 2021
@ascherer
Fix issue #4 with temporary patch.
664b33f
ascherer added a commit that referenced this issue May 8, 2022
@ascherer
Fix issue #4 with temporary patch.
0ab7d0b
@ascherer
Copy link
Owner Author

Fixed in Stanford GraphBase 2021-01-30.

ascherer added a commit that referenced this issue Apr 24, 2024
@ascherer
Fix issue #4 with temporary patch.
120a296
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ascherer