[PATCH] D26120: [Cortex-M0] Atomic lowering

Weiming Zhao via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 31 15:38:02 PDT 2016


weimingz added a comment.

Agree with the change in shouldExpandAtomicRMWInIR but for  ARMSubtarget::enableAtomicExpand(), 
changing from

  return hasAnyDataBarrier() && (!isThumb() || hasV8MBaselineOps()

to

  return hasAnyDataBarrier();

Is it too relax? Seems OK but not very sure.


Repository:
  rL LLVM

https://reviews.llvm.org/D26120





More information about the llvm-commits mailing list