This is an archive of the discontinued LLVM Phabricator instance.

Disable use of SCC bit from asm
ClosedPublic

Authored by rampitec on Apr 7 2021, 2:06 PM.

Diff Detail

Event Timeline

rampitec created this revision.Apr 7 2021, 2:06 PM
rampitec requested review of this revision.Apr 7 2021, 2:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 7 2021, 2:06 PM
arsenm added a comment.Apr 7 2021, 2:10 PM

Message / name is somewhat confusing since I assumed this meant the scc register

llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
4097–4098

I don't think this should be using SMLoc to check the string content. Isn't there an already parsed token?

rampitec retitled this revision from Disable use of SCC from asm to Disable use of SCC bit from asm.Apr 7 2021, 2:14 PM

Message / name is somewhat confusing since I assumed this meant the scc register

Retitled.

llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
4097–4098

Not since I did a function to parse all these bits together and in any order. I now have to search for it. Anyhow, it is just moved from the place below.

t-tye added inline comments.Apr 7 2021, 3:14 PM
llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
4099

Could the GPU name be printed? I think that is done in other error messages?

rampitec added inline comments.Apr 7 2021, 3:17 PM
llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
4099

Asm parser universally prints "on this GPU". If we want to be specific we probably need to update it everywhere.

t-tye accepted this revision.Apr 7 2021, 3:19 PM

LGTM

This revision is now accepted and ready to land.Apr 7 2021, 3:19 PM
This revision was automatically updated to reflect the committed changes.