This is an archive of the discontinued LLVM Phabricator instance.

[MachineLICM] Debug intrinsics shouldn't affect hoist decisions
ClosedPublic

Authored by gberry on Apr 30 2018, 1:25 PM.

Details

Summary

When checking if an instruction stores to a given frame index, check
that the instruction can write to memory before looking at the memory
operands list to avoid e.g. DBG_VALUE instructions that reference a
frame index preventing a load from that index from being hoisted.

Diff Detail

Repository
rL LLVM

Event Timeline

gberry created this revision.Apr 30 2018, 1:25 PM
This revision is now accepted and ready to land.May 3 2018, 4:29 PM
This revision was automatically updated to reflect the committed changes.