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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a reviewer: jdoerfert.
lebedev.ri added a comment.

> If an intrinsic is defined without outputs, but having side effects, it still can be removed completely from the program

This kinda sounds slightly backwards.
*If* there are side-effects, then i'd expect it not to be dropped regardless of `readnone` status.


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

https://reviews.llvm.org/D64414