[PATCH] D137937: [TableGen] Represent IntrHasSideEffects using inaccessiblemem read+write

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 14 06:34:46 PST 2022


foad added a comment.

> Map IntrHasSideEffects to a read+write of inaccessible memory, **which is the usual way to represent a side effect that cannot be modelled more precisely**. [emphasis mine]

Is it? Does that mean that things-with-side-effects only interact with other things-with-side-effects? Is that really the intention of the hasSideEffects flag on an intrinsic?


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

https://reviews.llvm.org/D137937



More information about the llvm-commits mailing list