From: Kugan <kugan.vivekanandarajah@linaro.org>
To: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Cc: James Greenhalgh <james.greenhalgh@arm.com>,
Marcus Shawcroft <Marcus.Shawcroft@arm.com>
Subject: [AARCH64] Add missing entries in iterator vwcore
Date: Thu, 01 Oct 2015 20:41:00 -0000 [thread overview]
Message-ID: <560D9A70.60404@linaro.org> (raw)
[-- Attachment #1: Type: text/plain, Size: 1288 bytes --]
Hi,
In "aarch64_get_lane<mode>" operand 0 is VEL, so for %<vwcore>0,
iterator vwcore should (?) support all the modes in VEL.
Ran into following error with a local patch for an existing test case.
However it can also be reproduced with the attached test case.
fnction âfn1â:
t.c:25:1: internal compiler error: output_operand: invalid %-code
}
^
0x8198fb output_operand_lossage(char const*, ...)
../../base/gcc/final.c:3417
0x81a45b output_asm_insn(char const*, rtx_def**)
../../base/gcc/final.c:3782
0x81b9d3 output_asm_insn(char const*, rtx_def**)
../../base/gcc/final.c:2364
0x81b9d3 final_scan_insn(rtx_insn*, _IO_FILE*, int, int, int*)
../../base/gcc/final.c:3029
0x81be2b final(rtx_insn*, _IO_FILE*, int)
../../base/gcc/final.c:2058
0x81c6e7 rest_of_handle_final
../../base/gcc/final.c:4449
0x81c6e7 execute
../../base/gcc/final.c:4524
Attached patch fixes this. Bootstrapped and regression tested for
aarch64-none-linux-gnu with no new regression. Is this OK for trunk?
Thanks,
Kugan
gcc/ChangeLog:
2015-10-02 Kugan Vivekanandarajah <kuganv@linaro.org>
* config/aarch64/iterators.md: Add missing core element mode for
mode.
gcc/testsuite/ChangeLog:
2015-10-02 Kugan Vivekanandarajah <kuganv@linaro.org>
* gcc.target/aarch64/foo.c: New test.
[-- Attachment #2: p.txt --]
[-- Type: text/plain, Size: 1319 bytes --]
diff --git a/gcc/config/aarch64/iterators.md b/gcc/config/aarch64/iterators.md
index 38c5a24..e49abd5 100644
--- a/gcc/config/aarch64/iterators.md
+++ b/gcc/config/aarch64/iterators.md
@@ -537,8 +537,11 @@
(V4HI "w") (V8HI "w")
(V2SI "w") (V4SI "w")
(DI "x") (V2DI "x")
+ (V4HF "w") (V8HF "w")
(V2SF "w") (V4SF "w")
- (V2DF "x")])
+ (V2DF "x") (SI "x")
+ (HI "x") (QI "x")])
+
;; Double vector types for ALLX.
(define_mode_attr Vallxd [(QI "8b") (HI "4h") (SI "2s")])
diff --git a/gcc/testsuite/gcc.target/aarch64/foo.c b/gcc/testsuite/gcc.target/aarch64/foo.c
index e69de29..77f161e 100644
--- a/gcc/testsuite/gcc.target/aarch64/foo.c
+++ b/gcc/testsuite/gcc.target/aarch64/foo.c
@@ -0,0 +1,25 @@
+
+/* { dg-do compile } */
+/* { dg-options "-O3" } */
+
+void fn2 ();
+
+typedef __Float16x4_t float16x4_t;
+__fp16 result_float16x4[1];
+float16x4_t exec_vst1_lane_vector_float16x4, exec_vst1_lane___trans_tmp_1;
+
+void fn1 ()
+{
+ exec_vst1_lane_vector_float16x4 = exec_vst1_lane___trans_tmp_1;
+ __fp16 *__a = result_float16x4;
+ float16x4_t __b = exec_vst1_lane___trans_tmp_1;
+ int __lane = 0;
+ *__a = ({ __b[__lane]; });
+ union {
+ short i;
+ __fp16 f;
+ } tmp_res;
+ tmp_res.f = result_float16x4[0];
+ if (tmp_res.i)
+ fn2();
+}
next reply other threads:[~2015-10-01 20:41 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-01 20:41 Kugan [this message]
2015-10-05 10:33 ` James Greenhalgh
2015-10-06 1:40 ` Kugan
2015-10-06 8:15 ` James Greenhalgh
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=560D9A70.60404@linaro.org \
--to=kugan.vivekanandarajah@linaro.org \
--cc=Marcus.Shawcroft@arm.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=james.greenhalgh@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).