[PATCH] D58233: Allow replacing intrinsic operands with variables

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D58233#1399047 <https://reviews.llvm.org/D58233#1399047>, @craig.topper wrote:

> What protects target intrinsics that haven't been annotated with immarg yet?


Nothing, these need to be accurate. I looked through the current set of in-tree target intrinsics, but don't see any comments or documentation stating certain arguments need to be constant so I hope we are OK there.


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

https://reviews.llvm.org/D58233