[PATCH] D66318: [SDAG] Update generic code to conservatively check for isAtomic in addition to isVolatile

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 10 11:36:39 PDT 2019


reames added a comment.

Porting a comment from craig.topper over on D66322 <https://reviews.llvm.org/D66322>:
"Will you also be updating nonvolatile_load/nonvolatile_store in include/llvm/Target/TargetSelectionDAG.td. We use those in several places where we change the size of a load."

Needs addressed in rebase.


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

https://reviews.llvm.org/D66318





More information about the llvm-commits mailing list