← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/atomic/gcc_traits.h

  • Committer: Brian Aker
  • Date: 2009-05-23 17:13:03 UTC
  • mfrom: (1034.1.8 merge)
  • Revision ID: brian@gaz-20090523171303-d28xhutqic0xe2b4
Merge Brian

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/atomic/gcc_traits.h
32
32
 
33
33
  gcc_traits() {}
34
34
 
35
 
  inline value_type add_and_fetch(volatile value_type *value, D addend )
 
35
  /* YES. I know these are semantically backwards...
 
36
   * so... TODO: Ensure we're doing the "right" thing here
 
37
   */
 
38
  inline value_type fetch_and_add(volatile value_type *value, D addend )
36
39
  {
37
40
    return __sync_add_and_fetch(value, addend);
38
41
  }
39
42
 
40
 
  inline value_type fetch_and_add(volatile value_type *value, D addend )
41
 
  {
42
 
    return __sync_fetch_and_add(value, addend);
43
 
  }
44
 
 
45
43
  inline value_type fetch_and_increment(volatile value_type *value)
46
44
  {
47
 
    return __sync_fetch_and_add(value, 1);
 
45
    return __sync_add_and_fetch(value, 1);
48
46
  }
49
47
 
50
48
  inline value_type fetch_and_decrement(volatile value_type *value)
51
49
  {
52
 
    return __sync_fetch_and_sub(value, 1);
 
50
    return __sync_sub_and_fetch(value, 1);
53
51
  }
54
52
 
55
53
  inline value_type fetch_and_store(volatile value_type *value,
56
54
                                    value_type new_value)
57
55
  {
 
56
    /* TODO: Is this the right one? */
58
57
    return __sync_lock_test_and_set(value, new_value);
59
58
  }
60
59
 
61
 
  inline bool compare_and_swap(volatile value_type *value,
 
60
  inline value_type compare_and_swap(volatile value_type *value,
62
61
                                     value_type new_value,
63
62
                                     value_type comparand )
64
63
  {
65
 
    return __sync_bool_compare_and_swap(value, comparand, new_value);
 
64
    return __sync_val_compare_and_swap(value, comparand, new_value);
66
65
  }
67
66
 
68
67
  inline value_type fetch(const volatile value_type *value) const volatile
69
68
  {
70
 
    /* 
71
 
     * This is necessary to ensure memory barriers are respected when
72
 
     * simply returning the value pointed at.  However, this does not
73
 
     * compile on ICC.
74
 
     *
75
 
     * @todo
76
 
     *
77
 
     * Look at how to rewrite the below to something that ICC feels is
78
 
     * OK and yet respects memory barriers.
79
 
     */
80
 
    return __sync_fetch_and_add(const_cast<value_type *>(value), 0);
 
69
    return __sync_add_and_fetch(const_cast<value_type *>(value), 0);
81
70
  }
82
71
 
83
72
  inline value_type store_with_release(volatile value_type *value,