[PATCH] D28742: [InstCombine] Don't DSE across readnone functions that may throw

[David Majnemer via Phabricator via llvm-commits]

majnemer added a comment.

Instead of using `I->mayReadFromMemoryOrThrow() || I->mayWriteToMemory()`, I'd recommend using `I->mayReadFromMemory() || I->mayHaveSideEffects()`

`mayReadFromMemoryOrThrow` is a rather strange predicate...


https://reviews.llvm.org/D28742