[PATCH] D78928: fp16 indexed patterns V16-V31 registers test cases.

Pavel Iliin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 27 07:29:37 PDT 2020


ilinpv created this revision.
ilinpv added reviewers: eli.friedman, ab, dmgreen.
Herald added subscribers: llvm-commits, arphaman, kristof.beyls.
Herald added a project: LLVM.

Test cases added as @eli.friedman kindly advised.
I am wondering do we really need them and need them in asm clobber form.
The questions originated from initial commit  be881e283173 <https://reviews.llvm.org/rGbe881e2831735d6879ee43710f5a4d1c8d50c615> with pre-commit code review D78252 <https://reviews.llvm.org/D78252> approved.
Then thanks to @ab notes I committed fixes to first parts of patterns(V16-V31 registers restrictions), although 
all used v*i16_indexed patterns in second part had already V16-V31 restrictions in place and no issues were found. 
Restrictions are tested by llvm/test/MC/AArch64/fullfp16-diagnostics.s
So test cases under review just additionally check that V16-V31 are not allocated in f16 indexed patterns.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D78928

Files:
  llvm/test/CodeGen/AArch64/fp16_intrinsic_lane.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78928.260317.patch
Type: text/x-patch
Size: 31946 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200427/f895ab67/attachment.bin>


More information about the llvm-commits mailing list