Intrinsic with sideeffect is optimized out

LLVM Project
6
  1. IntrHasSideEffects is a backend thing. It is only used there IIRC, and for the LLVM-IR it doesn’t make a difference.
  2. I would very much like to have more than just memory effects modeled in LLVM-IR. We have multiple such instances already. As @nikic finishes the memory attribute we can revisit it and maybe rename it into sideeffect, see for example [RFC] Unify memory effect attributes - #5 by jdoerfert