[PATCH] D28741: [EarlyCSE] Don't DSE across readnone functions that may throw

David Majnemer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 17 11:05:23 PST 2017


majnemer accepted this revision.
majnemer added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Transforms/Scalar/EarlyCSE.cpp:764-766
     // If this instruction may read from memory, forget LastStore.
     // Load/store intrinsics will indicate both a read and a write to
     // memory.  The target may override this (e.g. so that a store intrinsic
----------------
This comment probably needs a small tweak.


https://reviews.llvm.org/D28741





More information about the llvm-commits mailing list