This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU][MC][GFX11][NFC] Correct VOPD parsing
ClosedPublic

Authored by dp on Sep 8 2022, 7:08 AM.

Details

Summary

Instruction opcode for the Y operation is currently stored as an expression operand. This may cause incorrect detection of error position by getLitLoc. Currently, it does not because VOPD instructions are not validated by the parser.

This change is required for VOPD validation which is WIP.

Diff Detail

Event Timeline

dp created this revision.Sep 8 2022, 7:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 8 2022, 7:08 AM
dp requested review of this revision.Sep 8 2022, 7:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 8 2022, 7:08 AM
Joe_Nash accepted this revision.Sep 8 2022, 8:12 AM

LGTM

This revision is now accepted and ready to land.Sep 8 2022, 8:12 AM
This revision was landed with ongoing or failed builds.Sep 9 2022, 3:03 AM
This revision was automatically updated to reflect the committed changes.