mbox series

[0/4] fix testsuite on 32-bit

Message ID 20180909235327.26252-1-luc.vanoostenryck@gmail.com (mailing list archive)
Headers show
Series fix testsuite on 32-bit | expand

Message

Luc Van Oostenryck Sept. 9, 2018, 11:53 p.m. UTC
This series contains fixes for the three tests that failed
on 32-bit. It also contains a small improvement in the SSA
conversion which is related to one of the failed test.

Many thanks to Ramsay Jones for testing and reporting
these problems.


This series is available for review & testing in the Git repository at:
  git://github.com/lucvoo/sparse-dev.git fix-32bit

----------------------------------------------------------------
Luc Van Oostenryck (4):
      test: make test Waddress-space-strict succeed on 32-bit
      test: use integers of different sizes, even on 32-bit
      test: make 32-bit version of failed test
      ssa: relax what can be promoted

 ssa.c                                              |  4 +--
 validation/Waddress-space-strict.c                 | 33 +++++-----------------
 .../mem2reg/{init-local.c => init-local32.c}       |  4 +--
 validation/mem2reg/init-local64.c                  | 27 ++++++++++++++++++
 validation/optim/cse-size.c                        |  4 +--
 5 files changed, 40 insertions(+), 32 deletions(-)
 rename validation/mem2reg/{init-local.c => init-local32.c} (71%)
 create mode 100644 validation/mem2reg/init-local64.c

Comments

Ramsay Jones Sept. 15, 2018, 11:19 p.m. UTC | #1
On 10/09/18 00:53, Luc Van Oostenryck wrote:
> This series contains fixes for the three tests that failed
> on 32-bit. It also contains a small improvement in the SSA
> conversion which is related to one of the failed test.

Sorry for the late reply, but I finally found time to test
this series on 32bit (and 64bit) Linux. You will not be
surprised to hear that it fixes all reported issues.

Thanks!

ATB,
Ramsay Jones
Luc Van Oostenryck Sept. 24, 2018, 2:37 p.m. UTC | #2
On Sun, Sep 16, 2018 at 12:19:33AM +0100, Ramsay Jones wrote:
> 
> 
> On 10/09/18 00:53, Luc Van Oostenryck wrote:
> > This series contains fixes for the three tests that failed
> > on 32-bit. It also contains a small improvement in the SSA
> > conversion which is related to one of the failed test.
> 
> Sorry for the late reply, but I finally found time to test
> this series on 32bit (and 64bit) Linux. You will not be
> surprised to hear that it fixes all reported issues.

Many thanks for confirming this.

-- Luc