User account creation filtered due to spam.

Bug 78421 - [7/8 Regression] vect-strided-a-u8-i2-gap.c fails on armeb
Summary: [7/8 Regression] vect-strided-a-u8-i2-gap.c fails on armeb
Status: REOPENED
Alias: None
Product: gcc
Classification: Unclassified
Component: testsuite (show other bugs)
Version: 7.0
: P4 normal
Target Milestone: 7.2
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-18 13:10 UTC by Christophe Lyon
Modified: 2017-05-02 15:56 UTC (History)
3 users (show)

See Also:
Host:
Target: armeb
Build:
Known to work:
Known to fail:
Last reconfirmed: 2017-01-20 00:00:00


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Christophe Lyon 2016-11-18 13:10:49 UTC
Since r241991, gcc.dg/vect/vect-strided-a-u8-i2-gap.c fails on armeb:
  gcc.dg/vect/vect-strided-a-u8-i2-gap.c -flto -ffat-lto-objects  scan-tree-dump-times vect "vectorized 2 loops" 1
  gcc.dg/vect/vect-strided-a-u8-i2-gap.c scan-tree-dump-times vect "vectorized 2 loops" 1


This was discussed in the thread starting at:
https://gcc.gnu.org/ml/gcc-patches/2016-11/msg00550.html
Comment 1 Nick Clifton 2017-01-20 16:34:49 UTC
Hi Guys,

  I have proposed an extended version of Richard's patch to fix this regression:

https://gcc.gnu.org/ml/gcc-patches/2017-01/msg01615.html

Cheers
  Nick
Comment 2 Nick Clifton 2017-01-23 12:25:07 UTC
Author: nickc
Date: Mon Jan 23 12:24:35 2017
New Revision: 244796

URL: https://gcc.gnu.org/viewcvs?rev=244796&root=gcc&view=rev
Log:
	PR testsuite/78421
	* lib/target-supports.exp (check_effective_target_vect_hw_misalign):
	If the target is ARM return the result of the
	check_effective_target_arm_vect_no_misalign proc.
	* gcc.dg/vect/vect-strided-a-u8-i2-gap.c: If the target does not
	support unaligned vectors then only expect one of the loops to be
	unrolled.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gcc.dg/vect/vect-strided-a-u8-i2-gap.c
    trunk/gcc/testsuite/lib/target-supports.exp
Comment 3 Nick Clifton 2017-01-23 12:25:51 UTC
Patch applied.
Comment 4 ktkachov 2017-02-07 09:45:01 UTC
I still see the test failing on arm-none-eabi when testing with, for example:
-mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard

Christophe also has a report: https://gcc.gnu.org/ml/gcc-patches/2017-01/msg01806.html
Comment 5 Richard Biener 2017-03-14 11:35:01 UTC
(In reply to ktkachov from comment #4)
> I still see the test failing on arm-none-eabi when testing with, for example:
> -mthumb/-march=armv8-a/-mfpu=crypto-neon-fp-armv8/-mfloat-abi=hard
>
> Christophe also has a report:
> https://gcc.gnu.org/ml/gcc-patches/2017-01/msg01806.html

Please have a look at the vectorizer logs to see why it fails.

armeb is not primary or secondary target, demoting.
Comment 6 Jakub Jelinek 2017-05-02 15:56:14 UTC
GCC 7.1 has been released.