[PATCH] D66322: [X86] Updated target specific selection dag code to conservatively check for isAtomic in addition to isVolatile

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 1 02:33:04 PDT 2019


RKSimon added a comment.

We have the isSimple() wrapper for  in LoadInst and StoreInst, could we add that to MemSDNode as well?

  bool isSimple() const { return !isAtomic() && !isVolatile(); }


Repository:
  rL LLVM

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

https://reviews.llvm.org/D66322





More information about the llvm-commits mailing list