This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PR80101: Fix ICE in store_data_bypass_p
- From: Eric Botcazou <ebotcazou at adacore dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: gcc-patches at gcc dot gnu dot org, Kelvin Nilsen <kdnilsen at linux dot vnet dot ibm dot com>
- Date: Fri, 07 Apr 2017 10:39:03 +0200
- Subject: Re: [PATCH] PR80101: Fix ICE in store_data_bypass_p
- Authentication-results: sourceware.org; auth=none
- References: <ceb839ae-0101-11d2-d5ce-b402c7dcfe51@linux.vnet.ibm.com> <3316696.3QEehbYcbO@polaris> <20170407074824.GU4402@gate.crashing.org>
> Or we could just change "blockage" and wait for the next bug report.
That's my suggestion, yes.
> Alternatively, we can arrange for the bypass functions to not ICE. We
> can do that specific to these rs6000 pipeline descriptions, by having
> our own version of store_data_bypass_p; or we can make that function
> work for all insns (its definition works fine for insn pairs where
> not both the producer and consumer are SETs). That's what Kelvin's
> patch does. What is the value in ICEing here?
Telling the back-end writer that something may be wrong somewhere instead of
silently accepting nonsense? How long have all the assertions been there?
--
Eric Botcazou