This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Fix incomplete asm statement parsing
ClosedPublic

Authored by luismarques on Jan 18 2022, 7:23 AM.

Details

Summary

For instructions without operands, the final AsmToken::EndOfStatement wasn't being consumed. In the context of inline assembly, the resulting empty statements would cause extraneous empty lines to be emitted. Fix the issue by consuming the EndOfStatement token.

Diff Detail

Event Timeline

luismarques created this revision.Jan 18 2022, 7:23 AM
luismarques requested review of this revision.Jan 18 2022, 7:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 18 2022, 7:23 AM
asb accepted this revision.Jan 18 2022, 7:28 AM

Nice catch, thanks.

This revision is now accepted and ready to land.Jan 18 2022, 7:28 AM
This revision was landed with ongoing or failed builds.Jan 19 2022, 1:56 PM
This revision was automatically updated to reflect the committed changes.