[PATCH] D157389: AutoUpgrade: Use syncscope("agent") atomic.inc/dec intrinsic upgrade

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D157389#4569730 <https://reviews.llvm.org/D157389#4569730>, @sameerds wrote:

> What is the "old syncscope parameter" mentioned in the change description? In the actual change and the lit tests, the default "system" scope is being changed to "agent", which superficially looks like an incorrect change.

In the deleted intrinsic. There was an integer syncscope argument which was never actually used. This just needs to try emulating the old intrinsic behavior


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

https://reviews.llvm.org/D157389