On Feb 9, 2018, Jeff Law <l...@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
