This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [SFN+LVU+IEPM v4 9/9] [IEPM] Introduce inline entry point markers
- From: Alexandre Oliva <aoliva at redhat dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: Alan Modra <amodra at gmail dot com>, Jakub Jelinek <jakub at redhat dot com>, Jason Merrill <jason at redhat dot com>, Richard Biener <richard dot guenther at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 09 Feb 2018 08:34:08 -0200
- Subject: Re: [SFN+LVU+IEPM v4 9/9] [IEPM] Introduce inline entry point markers
- Authentication-results: sourceware.org; auth=none
- References: <orlgjec1b5.fsf_-_@lxoliva.fsfla.org> <20171110023448.28164-9-aoliva@redhat.com> <or7etsvep6.fsf@lxoliva.fsfla.org> <20180124171232.GG2063@tucnak> <orh8r94s88.fsf@lxoliva.fsfla.org> <orlgg2etl4.fsf@lxoliva.fsfla.org> <20180209035334.GF3846@bubble.grove.modra.org> <6e28e34e-681d-bebb-d815-569eee148de5@redhat.com>
On Feb 9, 2018, Jeff Law <law@redhat.com> wrote:
> On 02/08/2018 08:53 PM, Alan Modra wrote:
>> On Fri, Feb 09, 2018 at 01:21:27AM -0200, Alexandre Oliva wrote:
>>> Here's what I checked in, right after the LVU patch.
>>>
>>> [IEPM] Introduce inline entry point markers
>>
>> One of these two patches breaks ppc64le bootstrap with the assembler
>> complaining "Error: view number mismatch" when compiling
>> libdecnumber.
>>
> I've just passed along a similar failure (.i, .s and command line
> options) to Alex for ppc64 (be) building glibc.
This fixes at least the testcase Jeff provided me with. I'm going ahead
and checking it in as obvious. I suppose we might need more of these,
on this and other ports, if they have been sloppy about zero-length
pseudo insns :-(
Would you guys please let me know whether you still see a problem, if
you get a chance to respin? I was just about to crash in bed when I saw
your email.
When I get back up, I'll build the latest binutils release on ppc64,
ppc64el and aarch64, and then bootstrap gcc with it. I should have done
that when I broadened my testing of the SFN+LVU+IEPM patchset to those
platforms, but I didn't realize I was failing to test them with an
assembler with view support, doh! Sorry about that.
for gcc/ChangeLog
* config/rs6000/rs6000.md (blockage): Set length to zero.
diff --git a/gcc/config/rs6000/rs6000.md b/gcc/config/rs6000/rs6000.md
index 33f0d959f5d0..8aa4e0e7c71e 100644
--- a/gcc/config/rs6000/rs6000.md
+++ b/gcc/config/rs6000/rs6000.md
@@ -11063,7 +11063,8 @@
(define_insn "blockage"
[(unspec_volatile [(const_int 0)] UNSPECV_BLOCK)]
""
- "")
+ ""
+ [(set_attr "length" "0")])
(define_expand "probe_stack_address"
[(use (match_operand 0 "address_operand"))]
--
Alexandre Oliva, freedom fighter http://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/ FSF Latin America board member
Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer