Re: [PATCH 0/4] Properly handle GCC target("march=") (PR71652)

2016-09-22 Thread jh 

OK for patches 1-4.

Honza

Dne 2016-08-12 14:15, Martin Liška napsal:

PING^1


Hello.

Following small patch set targets $subject, where we ICE if someone
uses #pragma GCC target ("arch=generic"). My attempt is to not to
create a new target optimization node in case of a wrong value
of march string. Such approach does not generate multiple errors.

Apart from that, I also improved i386 option handling as mentioned
in: [1]

Patch bootstraps and survives regression tests on 
powerpc64le-unknown-linux-gnu.


Ready for trunk?
Thanks,
Martin

[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71652#c4

marxin (4):
  Fix PR target/71652
  Support crc32 as a i386 target optimization node
  Support movbe as a i386 target optimization node
  Remove fused-madd from documentation

 gcc/config/i386/i386.c| 64 
++-

 gcc/doc/extend.texi   |  5 ---
 gcc/testsuite/gcc.target/i386/crc32-5.c   | 25 
 gcc/testsuite/gcc.target/i386/movbe-4.c   | 20 ++
 gcc/testsuite/gcc.target/i386/pr71652-2.c | 13 +++
 gcc/testsuite/gcc.target/i386/pr71652-3.c | 14 +++
 gcc/testsuite/gcc.target/i386/pr71652.c   | 13 +++
 7 files changed, 130 insertions(+), 24 deletions(-)
 create mode 100644 gcc/testsuite/gcc.target/i386/crc32-5.c
 create mode 100644 gcc/testsuite/gcc.target/i386/movbe-4.c
 create mode 100644 gcc/testsuite/gcc.target/i386/pr71652-2.c
 create mode 100644 gcc/testsuite/gcc.target/i386/pr71652-3.c
 create mode 100644 gcc/testsuite/gcc.target/i386/pr71652.c

Re: [PATCH 0/4] Properly handle GCC target("march=") (PR71652)

2016-08-12 Thread Martin Liška 
PING^1

> Hello.
> 
> Following small patch set targets $subject, where we ICE if someone
> uses #pragma GCC target ("arch=generic"). My attempt is to not to
> create a new target optimization node in case of a wrong value
> of march string. Such approach does not generate multiple errors.
> 
> Apart from that, I also improved i386 option handling as mentioned
> in: [1]
> 
> Patch bootstraps and survives regression tests on 
> powerpc64le-unknown-linux-gnu.
> 
> Ready for trunk?
> Thanks,
> Martin
> 
> [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71652#c4
> 
> marxin (4):
>   Fix PR target/71652
>   Support crc32 as a i386 target optimization node
>   Support movbe as a i386 target optimization node
>   Remove fused-madd from documentation
> 
>  gcc/config/i386/i386.c| 64 
> ++-
>  gcc/doc/extend.texi   |  5 ---
>  gcc/testsuite/gcc.target/i386/crc32-5.c   | 25 
>  gcc/testsuite/gcc.target/i386/movbe-4.c   | 20 ++
>  gcc/testsuite/gcc.target/i386/pr71652-2.c | 13 +++
>  gcc/testsuite/gcc.target/i386/pr71652-3.c | 14 +++
>  gcc/testsuite/gcc.target/i386/pr71652.c   | 13 +++
>  7 files changed, 130 insertions(+), 24 deletions(-)
>  create mode 100644 gcc/testsuite/gcc.target/i386/crc32-5.c
>  create mode 100644 gcc/testsuite/gcc.target/i386/movbe-4.c
>  create mode 100644 gcc/testsuite/gcc.target/i386/pr71652-2.c
>  create mode 100644 gcc/testsuite/gcc.target/i386/pr71652-3.c
>  create mode 100644 gcc/testsuite/gcc.target/i386/pr71652.c
>

[PATCH 0/4] Properly handle GCC target("march=") (PR71652)

2016-07-18 Thread marxin 
Hello.

Following small patch set targets $subject, where we ICE if someone
uses #pragma GCC target ("arch=generic"). My attempt is to not to
create a new target optimization node in case of a wrong value
of march string. Such approach does not generate multiple errors.

Apart from that, I also improved i386 option handling as mentioned
in: [1]

Patch bootstraps and survives regression tests on powerpc64le-unknown-linux-gnu.

Ready for trunk?
Thanks,
Martin

[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71652#c4

marxin (4):
  Fix PR target/71652
  Support crc32 as a i386 target optimization node
  Support movbe as a i386 target optimization node
  Remove fused-madd from documentation

 gcc/config/i386/i386.c| 64 ++-
 gcc/doc/extend.texi   |  5 ---
 gcc/testsuite/gcc.target/i386/crc32-5.c   | 25 
 gcc/testsuite/gcc.target/i386/movbe-4.c   | 20 ++
 gcc/testsuite/gcc.target/i386/pr71652-2.c | 13 +++
 gcc/testsuite/gcc.target/i386/pr71652-3.c | 14 +++
 gcc/testsuite/gcc.target/i386/pr71652.c   | 13 +++
 7 files changed, 130 insertions(+), 24 deletions(-)
 create mode 100644 gcc/testsuite/gcc.target/i386/crc32-5.c
 create mode 100644 gcc/testsuite/gcc.target/i386/movbe-4.c
 create mode 100644 gcc/testsuite/gcc.target/i386/pr71652-2.c
 create mode 100644 gcc/testsuite/gcc.target/i386/pr71652-3.c
 create mode 100644 gcc/testsuite/gcc.target/i386/pr71652.c

-- 
2.8.4