[PATCH] D63938: [ARM] Stop using scalar FP instructions in integer-only MVE mode.
Simon Tatham via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 1 09:49:26 PDT 2019
simon_tatham updated this revision to Diff 207361.
simon_tatham added a comment.
Revised patch fixes those two target triples in the tests, and makes the `setAllExpand` calls for f32 and f64 conditional on different things. Also, to make that less cumbersome, I've moved a few re-enabling `setOperationAction` calls into `setAllExpand` itself which otherwise had to be run after every single call.
This version of the patch is intended to apply _before_ D63937 <https://reviews.llvm.org/D63937> rather than after it, because this way round it turned out easier to get the tests to pass in the intermediate state.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63938/new/
https://reviews.llvm.org/D63938
Files:
llvm/lib/Target/ARM/ARMISelLowering.cpp
llvm/lib/Target/ARM/ARMISelLowering.h
llvm/lib/Target/ARM/ARMInstrVFP.td
llvm/test/CodeGen/ARM/fp16-instructions.ll
llvm/test/CodeGen/Thumb2/float-ops.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63938.207361.patch
Type: text/x-patch
Size: 11125 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190701/2c29e686/attachment.bin>
More information about the llvm-commits
mailing list