[PATCH] D106309: [LLVM IR] Allow volatile stores to trap.

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

I think this makes it consistent but I would have hoped to avoid it anyway. `willreturn` might not be the most important attribute yet but it's uses are growing. It affects `align`, `dereferenceable`, and already various optimizations already.

That said, the conversation in the other thread doesn't have to stop if we land this and this would at least give us some consistency, so I'm in favor of landing it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106309