This is an archive of the discontinued LLVM Phabricator instance.

RISCV: Add support for floating point registers in inlineasm
ClosedPublic

Authored by simoncook on Jul 15 2019, 6:19 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

simoncook created this revision.Jul 15 2019, 6:19 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 15 2019, 6:19 AM
simoncook retitled this revision from [PATCH] RISCV: Add support for floating point registers in inlineasm to RISCV: Add support for floating point registers in inlineasm.Jul 15 2019, 6:19 AM
simoncook added a comment.EditedJul 15 2019, 6:41 AM

As an aside, I've noticed a codegen issue when using floating point clobber lists, resulting in the implicit-defs not being added to INLINEASM instructions. I'm working on a fix for that now and will submit a second patch shortly.

Update: This fix is D64751

lenary accepted this revision.Jul 23 2019, 5:13 AM

Looks good to me! Thank you!

This revision is now accepted and ready to land.Jul 23 2019, 5:13 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJul 31 2019, 2:11 AM