This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Fix bug with tracking processed blocks in SIInsertWaitcnts
ClosedPublic

Authored by scott.linder on Jun 20 2018, 1:08 PM.

Details

Summary

BlockWaitcntProcessedSet was not being cleared between calls, so it was producing incorrect counts in cases where MBB addresses happened to coincide across multiple calls.

I was not able to reproduce this consistently (depends on non-deterministic addresses), so I could not include a test.

Diff Detail

Repository
rL LLVM

Event Timeline

scott.linder created this revision.Jun 20 2018, 1:08 PM
This revision is now accepted and ready to land.Jun 20 2018, 1:28 PM
This revision was automatically updated to reflect the committed changes.