[llvm-dev] Changes to TableGen in v4.0?

[Nemanja Ivanovic via llvm-dev]

I believe the node you're looking for is fpround.
That one has a type profile with the source larger than the target. The
fround does not - they have to be the same.

I imagine this is a result of:
------------------------------------------------------------------------
r279129 | mkuper | 2016-08-18 15:08:15 -0500 (Thu, 18 Aug 2016) | 8 lines

[SelectionDAG] Rename fextend -> fpextend, fround -> fpround, frnd -> fround

The names of the tablegen defs now match the names of the ISD nodes.
This makes the world a slightly saner place, as previously "fround" matched
ISD::FP_ROUND and not ISD::FROUND.

Differential Revision: https://reviews.llvm.org/D23597

------------------------------------------------------------------------

On Mon, Jan 23, 2017 at 12:41 PM, Martin J. O'Riordan via llvm-dev <
llvm-dev at lists.llvm.org> wrote:

> I am trying to upgrade to the LLVM v4.0 branch, but I am seeing failures
> in my TableGen descriptions for conversion from FP32 to FP16 (scalar and
> vector).
>
>
>
> The patterns I have are along the lines of:
>
>
>
> [(set (f16 RF16:$dst), (fround (f32 RF32:$src)))]
>
> or:
>
> [(set (v2f16 VF16:$dst), (fround (v2f32 VF32:$src)))]
>
>
>
> and these now produce the errors:
>
>
>
> error: In CONV_f32_f16: Type inference contradiction found, merging 'f32'
> into 'f16'
>
> or:
>
> error: In CONV_v2f32_v2f16: Type inference contradiction found, merging
> 'v2f32' into 'v2f16'
>
>
>
> For the scalar version, the conflict is between two rules that have
> different mutually exclusive predicates (for alternative chip versions).
> But in the case of the vector, the conflict is for a vector of a different
> size, in the case:
>
>
>
> [(set (v4f16 VF16:$dst), (fround (v4f32 VF32:$src)))]
>
>
>
> and ‘CONV_v4f32_v4f16’.  What adjustments do I need to make to the TD
> descriptions to make these work again?  I know that FP16 is not hugely
> common on the mainstream platforms, but it is vital to ours.
>
>
>
> I did notice that ‘fextend’ was replaced by ‘fpextend’, is there some new
> ISD node type I should use for ‘fround’?
>
>
>
> Thanks,
>
>
>
>             MartinO
>
>
>
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20170123/66f90488/attachment.html>