This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Rename vector inline constraint from 'v' to 'vr' and 'vm' in IR.
ClosedPublic

Authored by HsiangKai on Jul 30 2021, 2:12 AM.

Diff Detail

Event Timeline

HsiangKai created this revision.Jul 30 2021, 2:12 AM
HsiangKai requested review of this revision.Jul 30 2021, 2:12 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJul 30 2021, 2:12 AM
craig.topper added inline comments.Jul 30 2021, 2:14 PM
clang/lib/Basic/Targets/RISCV.cpp
108

I believe most targets use std::string("^") + std::string(Constraint, 2) for 2 letter constraints

AArch64 added the @ format and uses @3 for a 3 letter constraint. But it didn't get documented in the LangRef.

Can we use the ^ here which is documented.

HsiangKai updated this revision to Diff 363241.Jul 30 2021, 5:47 PM

Use '^' instead of '@2'.

This revision is now accepted and ready to land.Jul 30 2021, 6:13 PM