Store Conditional can retire successfully even if atomics are disabled

[Timmmm]

/* NOTE: Currently, we only EA if address translation is successful. This may need revisiting. */
function clause execute (STORECON(aq, rl, rs2, rs1, width, rd)) = {
  if speculate_conditional () == false then {
    /* should only happen in rmem
     * rmem: allow SC to fail very early
     */
    X(rd) = zero_extend(0b1); RETIRE_SUCCESS
  } else {
    if haveAtomics() then {
      /* normal non-rmem case
       * rmem: SC is allowed to succeed (but might fail later)
       */

Surely speculate_conditional() should be inside if haveAtomics()?

[jordancarlin]

@Timmmm I think this was resolved by #469 as the whole execute clause will only be reached if haveAtomics() is true now.

[Timmmm]

Yes you're right. I think there are some other issues with it but i'll open a new issue for them.