This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU] Remove SIEncodingFamily.GFX10_B
ClosedPublic

Authored by rampitec on Sep 29 2020, 3:41 PM.

Details

Summary

It turns to be not needed anymore.

Diff Detail

Event Timeline

rampitec created this revision.Sep 29 2020, 3:41 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 29 2020, 3:41 PM
rampitec requested review of this revision.Sep 29 2020, 3:41 PM

Is there a missing c++ change? What was this for before?

Is there a missing c++ change? What was this for before?

The c++ value was removed long time ago, so it is out of sync with td. But this encoding is not used by any instructions for a long time too. This is just a remaining garbage after encoding uses were removed.

arsenm accepted this revision.Sep 29 2020, 3:55 PM
This revision is now accepted and ready to land.Sep 29 2020, 3:55 PM
This revision was automatically updated to reflect the committed changes.