[CDRIVER-616] __sync_add_and_fetch_4 missing on i386 Created: 13/Apr/15 Updated: 07/Oct/15 Resolved: 07/May/15 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.1.5 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Jonathan Reams | Assignee: | A. Jesse Jiryu Davis |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Description |
|
When on GCC 4.1 compiling for i386, we don't use the intrinsic version of __sync_add_and_fetch for 64-bit values, but do use the intrinsic of __sync_add_and_fetch for 32-bit values, which leads to a compile failure. I believe this is an extension of A workaround for this is to compile with "-march=i686", which turns on the compiler intrinsic. |
| Comments |
| Comment by Githook User [ 07/Oct/15 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Merge branch 'master' into 1.2.0-dev
Conflicts: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Githook User [ 07/Oct/15 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Merge branch 'master' into 1.2.0-dev
Conflicts: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Githook User [ 07/Oct/15 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Author: {u'username': u'ajdavis', u'name': u'A. Jesse Jiryu Davis', u'email': u'jesse@mongodb.com'}Message: Merge branch 'master' into 1.2.0-dev
Conflicts: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by A. Jesse Jiryu Davis [ 22/Apr/15 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
The hand-built logic to determine whether __sync_add_and_fetch is available, in bson-atomic.h, should be replaced with a configure-time check. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Comment by Jonathan Reams [ 13/Apr/15 ] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Actually, this is reproducible in the latest version of GCC.
|