On 22.11.18 20:56, Jakub Jelinek wrote: > Hi! > > This test apparently FAILs on s390x-linux, which is an effective target of > both logical_op_short_circuit and branch_cost. > The test has > /* { dg-additional-options "-mbranch-cost=2" { target branch_cost } } */ > and that option effectively makes the target ! logical_op_short_circuit, > but the effective target just reflects the default setting. So, I think > it should expect 0 ifs rather than 2 if -mbranch-cost=2 is used and > 2 when that option isn't used. > > Tested on x86_64-linux (which is a branch_cost but not > logical_op_short_circuit > effective target), Andreas, does this fix it on s390x-linux (I've only > looked at the optimized dump using a cross-compiler)?
Yes it does. Thanks for having a look! Andreas