[PATCH] D64414: Do not set ReadNone attribute on intrinsics with side effects

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 11:10:42 PDT 2019


chill added a comment.

In D64414#1576252 <https://reviews.llvm.org/D64414#1576252>, @jdoerfert wrote:

> I'm a bit lost. What intrinsic do we have that is "no-mem" but has side-effects?


E.g  `__builtin_arm_tcommit`, which I'm adding in this series, or  `__builtin_arm_irg` (which is *should* be `NoMem`, but is not, perhaps as a workaround for this issue).


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

https://reviews.llvm.org/D64414





More information about the llvm-commits mailing list