[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