diff options
author | Jingyue Wu <jingyue@google.com> | 2015-07-01 21:32:42 +0000 |
---|---|---|
committer | Jingyue Wu <jingyue@google.com> | 2015-07-01 21:32:42 +0000 |
commit | a0a56601c06293e078e5c4cfce08a7b60d4e88f3 (patch) | |
tree | 819e8cc8577863696e0dd28f45c5328f80bacc07 /llvm/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp | |
parent | f43a42d53e21e5860217ede634f8eea6d25d3a18 (diff) | |
download | llvm-a0a56601c06293e078e5c4cfce08a7b60d4e88f3.zip llvm-a0a56601c06293e078e5c4cfce08a7b60d4e88f3.tar.gz llvm-a0a56601c06293e078e5c4cfce08a7b60d4e88f3.tar.bz2 |
[NVPTX] expand extload/truncstore for vectors of floats
Summary:
According to PTX ISA:
For convenience, ld, st, and cvt instructions permit source and destination data operands to be wider than the instruction-type size, so that narrow values may be loaded, stored, and converted using regular-width registers. For example, 8-bit or 16-bit values may be held directly in 32-bit or 64-bit registers when being loaded, stored, or converted to other types and sizes. The operand type checking rules are relaxed for bit-size and integer (signed and unsigned) instruction types; floating-point instruction types still require that the operand type-size matches exactly, unless the operand is of bit-size type.
So, the ISA does not support load with extending/store with truncatation for floating numbers. This is reflected in setting the loadext/truncstore actions to expand in the code for floating numbers, but vectors of floating numbers are not taken care of.
As a result, loading a vector of floats followed by a fp_extend may be combined by DAGCombiner to a extload, and the extload may be lowered to NVPTXISD::LoadV2 with extending information. However, NVPTXISD::LoadV2 does not perform extending, and no extending instructions are inserted. Finally, PTX instructions with mismatched types are generated, like
ld.v2.f32 {%fd3, %fd4}, [%rd2]
This patch adds the correct actions for vectors of floats, so DAGCombiner would not create loads with extending, and correct code is generated.
Patched by Gang Hu.
Test Plan: Test case attached.
Reviewers: jingyue
Reviewed By: jingyue
Subscribers: llvm-commits, jholewinski
Differential Revision: http://reviews.llvm.org/D10876
llvm-svn: 241191
Diffstat (limited to 'llvm/lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp')
0 files changed, 0 insertions, 0 deletions