Remove COMPILER FENCE

poojanilangekar · poojanilangekar · commit ec41e65910e7 · 2018-02-21T15:14:50.000-05:00
diff --git a/src/concurrency/timestamp_ordering_transaction_manager.cpp b/src/concurrency/timestamp_ordering_transaction_manager.cpp
@@ -66,9 +66,6 @@ bool TimestampOrderingTransactionManager::SetLastReaderCommitId(
     // then set last_reader_cid to current_cid.
     if (*ts_ptr < current_cid) {
       *ts_ptr = current_cid;
-      // Since we need this memory address to be updated before other
-      // transactions attempt to modify/read the tuple.
-      COMPILER_MEMORY_FENCE;
     }
 
     GetSpinLatchField(tile_group_header, tuple_id)->Unlock();

Original file line number	Diff line number	Diff line change
`@@ -66,9 +66,6 @@ bool TimestampOrderingTransactionManager::SetLastReaderCommitId(`
`66`	`66`	`// then set last_reader_cid to current_cid.`
`67`	`67`	`if (*ts_ptr < current_cid) {`
`68`	`68`	`*ts_ptr = current_cid;`
`69`		`- // Since we need this memory address to be updated before other`
`70`		`- // transactions attempt to modify/read the tuple.`
`71`		`- COMPILER_MEMORY_FENCE;`
`72`	`69`	`}`
`73`	`70`
`74`	`71`	`GetSpinLatchField(tile_group_header, tuple_id)->Unlock();`