This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Set hasSideEffects 0 on _term instructions
ClosedPublic

Authored by arsenm on Mar 25 2019, 7:29 AM.

Details

Reviewers
rampitec
Summary

These were defaulting to true, but they are just wrappers around bit
operations. This avoids regressions in the exec mask optimization
passes in a future commit.

Diff Detail

Event Timeline

arsenm created this revision.Mar 25 2019, 7:29 AM
This revision is now accepted and ready to land.Mar 25 2019, 10:14 AM

One question though, did it pass precheckin? Make sure it did before submission.

arsenm closed this revision.Mar 25 2019, 2:09 PM

r356952