[PATCH] D76127: [TableGen] Do not set ReadOnly attribute on intrinsics with side effects

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

TBH, I feel "X is readonly and has side effects" sends the wrong message to begin with. It is a contradiction (in the IR world) as basically shown by the need for this patch. Given that there are no examples in-tree I don't understand why one would mark a side-effect intrinsic as readonly (or similar). Long story short, I would argue this should be a load error, not silently ignored.


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

https://reviews.llvm.org/D76127