[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