There is a line here that is no longer 100% true:

// Transactions do not survive term changes, so combining "getTerm" here with the
// recovery unit timestamp does not cause races.
_speculativeTransactionReadOpTime = {readTimestamp, replCoord->getTerm()};

This is no longer true for prepared transactions since they can survive term changes. I expect this is still safe since prepared transactions will do a write and not wait on the _speculativeTransactionReadOpTime but this should be double checked, and commented as such.

"OpTime creation" in the transaction participant is done here as well, very similarly.