On 02/01/2017 04:55 AM, Jakub Jelinek wrote:
On Wed, Feb 01, 2017 at 12:52:16PM +0100, Martin Liška wrote:
On 02/01/2017 11:38 AM, Jakub Jelinek wrote:
On Wed, Feb 01, 2017 at 11:34:48AM +0100, Martin Liška wrote:
Presumably the issue with print_operand_address is that there are paths where
s390_decompose_address can return without initializing AD/OUT. But AFAICT those
are invalid addresses that presumably shouldn't be showing up in
print_operand_address.
Can you add an assert in print_operand_address to ensure decomposition never
returns false?
Can't it happen e.g. with inline asm and "X" constraint?
output_operand_lossage then would emit an error rather than ICE for
something that is a user code bug, not internal compiler error.
Ok, thus said I'll commit the original version.
Is it fine?
Fine with me, but let Jeff chime in if he disagrees.
Sorry, didn't realize this stalled on me. Let's go with the original
patch, which I just installed for Martin as penance for slowing this
down :-)
jeff
