-
Notifications
You must be signed in to change notification settings - Fork 1
Error: std::bad_alloc in test-unordered on ppc32 #6
Comments
There are assumptions about sizes within the "partition" code; this would be the cause of the failure. Thanks for picking this one up. Probably affects Windows i386 (32-bit) too. |
@stephematician Thank you! Once there is a fix, I will test it. |
@barracuda156 - I've looked through the partition code; it's not clear to me why there's a bad alloc. I don't have a powerpc to debug on so looks like this one won't get fixed by me. |
@stephematician Do tests pass fine on 32-bit Linux (i.e., |
They pass on Windows i386 (32-bit). I have not tested building i386 R on Linux. Valgrind passes, and as far as i know, the ASAN tests on CRAN pass, too. If I had a stacktrace that included the library calls then I'd know where to start looking. |
How do I get it? Run a test binary via GDB? |
I think so; that's roughly how I debug on Ubuntu - e.g. I'll run I'm not the most experienced dev, so if you have any suggestions, let me know! |
@stephematician Sorry for a delay, I will return to this shortly. |
@barracuda156 - just letting you know I've tested with linux i386 (via Docker, i386/ubuntu image - stock standard build-essential kit) and there were no failures during the test. |
@stephematician Thank you! And thanks for reminding of the issue – it got lost in the queue of to-dos :( I gonna re-run tests and try to get meaningful output. |
UPD1. The error did not vanish with gcc13, just for the record. |
@stephematician I will test the updated version now. |
Hi - thanks, I've moved the project to gitlab: https://gitlab.com/stephematician/literanger - so stay tuned there for updates. |
P. S. This is not necessarily a problem with the code, since
malloc
errors show up once in a while when there are duplicatelibstdc++
(one belonging to the OS and another to GCC). However, forR
we have fixed that problem, and I do not see it now when testing packages. So possibly something assumes 64-bitness?The text was updated successfully, but these errors were encountered: