[PATCH] D109219: [Intrinsics] Move dest arg functions of MemIntrinsicBase deeper in mem intrinsics hierarchy

[Max Kazantsev via Phabricator via llvm-commits]

mkazantsev added a comment.

In D109219#2984715 <https://reviews.llvm.org/D109219#2984715>, @nikic wrote:

> This looks okay **if** we actually want to add new non-write intrinsics in this family (and it makes sense to have them in the same hierarchy). Is there a pending proposal for that?

I don't think it has been proposed yet, but we plan to introduce an [element-atomic] memcmp intrinsic. Generally, I think this change makes sense even without this notion, just because dest isn't placed where it belongs, and there is no legit way to express a read-only memory intrinsic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109219