[PATCH] D50786: [AST] Remove notion of volatile from alias sets

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

In https://reviews.llvm.org/D50786#1200886, @hiraditya wrote:

> There was discussion sometime back, might be worth looking at: https://groups.google.com/forum/#!topic/llvm-dev/uLvitfqEd_g


I'm missing why this is relevant?  It happens to involve a volatile access as an AliasSet, but seems otherwise unrelated.  From what I can tell, the "volatile implies mod" thing hasn't been true for years.


Repository:
  rL LLVM

https://reviews.llvm.org/D50786