Skip to content

Conversation

@ruevs
Copy link
Member

@ruevs ruevs commented Jun 26, 2022

@ruevs ruevs requested review from ppd and rpavlik June 26, 2022 12:00
@ruevs ruevs force-pushed the Win32LargeAddressAware branch from cfd368e to 5e7a6b0 Compare June 26, 2022 12:02
@ruevs
Copy link
Member Author

ruevs commented Jun 26, 2022

@rpavlik @ppd would you please take a look at this since it affects CMake and I am not exactly "proficient" in it :-)

@rpavlik
Copy link
Contributor

rpavlik commented Jun 26, 2022

Looks ok (though it would be better to use target_compile_options on the targets directly)

@ppd
Copy link
Member

ppd commented Jun 27, 2022

Looks good and is consistent with the rest of the cMake hodgepodge :)
The only issue I have is that the comment is the same for both toolchains:

Link 32 bit SolveSpace with /LARGEADDRESSAWARE

For MinGW, it links with the equivalent to /LARGEADRESSAWARE, but not exactly with it. That'd be my only nitpick.

@ruevs ruevs force-pushed the Win32LargeAddressAware branch from 5e7a6b0 to d864013 Compare June 30, 2022 10:25
@ruevs
Copy link
Member Author

ruevs commented Jun 30, 2022

it would be better to use target_compile_options on the targets directly

@rpavlik If I only knew what "target_compile_options" were in the context of cmake :-) Unfortunately I have no time to learn it.

The only issue I have is that the comment is the same for both toolchains:

@ppd Good catch - I fixed it and force pushed.

@rpavlik
Copy link
Contributor

rpavlik commented Jun 30, 2022

I'm more concerned about what Solvespace models you're using that need that much memory 🤣 Thanks for the good work!

@rpavlik rpavlik merged commit d6e1b23 into solvespace:master Jun 30, 2022
@ruevs
Copy link
Member Author

ruevs commented Jun 30, 2022

I'm more concerned about what Solvespace models you're using that need that much memory 🤣

@rpavlik not me :-) But it is pretty impressive what people are doing with the "new fast SolveSpace" :-)
See here #1261

@ruevs ruevs deleted the Win32LargeAddressAware branch June 30, 2022 13:24
@phkahler
Copy link
Member

BTW this doesn't actually "fix" the issue, it just pushes it out a bit further ;-) "Use 64bit Solvespace" is a nice workaround.

@ruevs ruevs added the Win32 label Jun 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

32bit SolveSpace crashes when it runs out of memory (was SolveSpace 3.1 is crashing so frequently without any warning/error on assembling parts.)

4 participants