[PATCH] D60679: [LowerAtomic] Lower fadd and fsub atomicrmw instructions

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 15 03:41:43 PDT 2019


arsenm added a comment.

This still needs tests, but it seems this duplicate pass thing could use some work. Both passes have essentially the same switch to turn the atomic into the equivalent non-atomic. It seems to be me like AtomicExpandPass is a superset of LowerAtomic, except AtomicExpand seems to ignore fences


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60679/new/

https://reviews.llvm.org/D60679





More information about the llvm-commits mailing list