[PATCH] D57825: IR: Add immarg attribute

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D57825#1426637 <https://reviews.llvm.org/D57825#1426637>, @spatel wrote:

> Changes here seem ok, but let me confirm: do we expect any changes/harm for targets that are not in the stack of dependent patches? Either way, reply to the llvm-dev thread to remind those targets and out-of-tree targets that this change has been committed (once it lands)?


Yes, but only after D58233 <https://reviews.llvm.org/D58233>. Until D58233 <https://reviews.llvm.org/D58233>, these aren't mandatory. After that, these need to be accurate or else passes may incorrectly replace arguments if these aren't accurate


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

https://reviews.llvm.org/D57825