This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

RE: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index

From: "Ian Bolton" <Ian dot Bolton at arm dot com>
To: "'Jakub Jelinek'" <jakub at redhat dot com>
Cc: "'Richard Guenther'" <richard dot guenther at gmail dot com>, <gcc-patches at gcc dot gnu dot org>
Date: Wed, 18 Aug 2010 11:51:44 +0100
Subject: RE: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
References: <4451172357808738623@unknownmsgid> <AANLkTi=JDQMS60QCwu1QWaOw6eAky7XX0mSsPLXEvxi0@mail.gmail.com> <-3980594348023223009@unknownmsgid> <AANLkTim+Vru5i23RQTvTS0OsPF+3ZtMbHtFeRh-kHfG+@mail.gmail.com> <8907875473453595467@unknownmsgid> <AANLkTi=3xS3zHGVATq8rbq_qsoxPUfW4AMoYap8T3ZLF@mail.gmail.com> <6239476478021296254@unknownmsgid> <AANLkTikRDYSxPZLa2xHmT_vP9G9VnYrUK8gqZmhjGZY6@mail.gmail.com> <000001cb3ec0$fc8dde70$f5a99b50$@Bolton@arm.com> <20100818104722.GO702@tyan-ft48-01.lab.bos.redhat.com>

> Tha formatting is wrong, no space should be after ( .

Well spotted.  Here's the amended version.

All OK now?

Cheers,
Ian


2010-08-18  Ian Bolton  <ian.bolton@arm.com>

	* tree-switch-conversion.c (gen_inbound_check): Ensure that the
	type for the conditional has wide enough range.

	* testsuite/g++.dg/pr44328.C: New test.

Attachment: pr44328.trunk.patch
Description: Binary data

Follow-Ups:
- Re: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
  - From: Martin Jambor

References:
- Re: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
  - From: H.J. Lu
- Re: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
  - From: H.J. Lu
- Re: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
  - From: H.J. Lu
- Re: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
  - From: Richard Guenther
- RE: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
  - From: Ian Bolton
- Re: [Patch] Fix PR44328 - switch/case optimization produces an invalid lookup table index
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]