[PATCH] D63437: [CodeGen][ARM] Fix FP16 vector coercion

2019-06-18 Thread Mikhail Maltsev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL363687: [CodeGen][ARM] Fix FP16 vector coercion (authored by miyuki, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D63437: [CodeGen][ARM] Fix FP16 vector coercion

2019-06-18 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard accepted this revision. ostannard added a comment. This revision is now accepted and ready to land. Ah, right. In that case, this LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63437/new/ https://reviews.llvm.org/D63437

[PATCH] D63437: [CodeGen][ARM] Fix FP16 vector coercion

2019-06-17 Thread Mikhail Maltsev via Phabricator via cfe-commits
miyuki added a comment. In D63437#1546312 , @ostannard wrote: > This doesn't look like the right pace to fix this - the backend can handle > vectors of i8 and i16, which are also not legal types, so why can't it > correctly handle vectors of f16?

[PATCH] D63437: [CodeGen][ARM] Fix FP16 vector coercion

2019-06-17 Thread Oliver Stannard (Linaro) via Phabricator via cfe-commits
ostannard added a comment. This doesn't look like the right pace to fix this - the backend can handle vectors of i8 and i16, which are also not legal types, so why can't it correctly handle vectors of f16? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D63437: [CodeGen][ARM] Fix FP16 vector coercion

2019-06-17 Thread Mikhail Maltsev via Phabricator via cfe-commits
miyuki created this revision. miyuki added reviewers: eli.friedman, olista01. Herald added subscribers: kristof.beyls, javed.absar. Herald added a project: clang. When a function argument or return type is a homogeneous aggregate which contains an FP16 vector but the target does not support FP16