LoadInst isn't enough; we need to include intrinsics that perform loads too.
All side-effecting intrinsics and such are already covered by the isSafe check, so we just need to care about things that read from memory.
Looks good to me with the minor tweak to the test. To keep the breadcrumb for future reference, I had changed this in D33179.
i32 %off or just get rid of the arguments entirely
This was merged in rL322311, closing to get it off my dashboard. In the future, ideally add
Differential Revision: https://reviews.llvm.org/D41960
with that syntax to the commit message, so Phabricator can close the revision automatically.