[PATCH] D16986: [LICM] Don't assert on volatile accesses

[Krzysztof Parzyszek via llvm-commits]

kparzysz added a subscriber: kparzysz.
kparzysz added a comment.

Regarding the potential AST modification:  loads from null or stores to it are not legal.  The original testcase isn't legal either (it contains a dereference of an uninitialized pointer).  While the compiler should not abort on any code, we should not be making special accommodations for handling such situations that could negatively affect legal cases.


http://reviews.llvm.org/D16986