[valgrind] [Bug 387712] s390x cgijnl reports Conditional jump or move depends on uninitialised value(s)

2017-12-12 Thread Mark Wielaard
https://bugs.kde.org/show_bug.cgi?id=387712

Mark Wielaard  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Mark Wielaard  ---
commit d6a810760ec61ddedf15445457edbbe288536a2f

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 387712] s390x cgijnl reports Conditional jump or move depends on uninitialised value(s)

2017-12-12 Thread Mark Wielaard
https://bugs.kde.org/show_bug.cgi?id=387712

--- Comment #4 from Mark Wielaard  ---
(In reply to Julian Seward from comment #3)
> This fixes it for me.  Mark, can you give it a spin?

Works for me. I ran the whole elfutils testsuite under valgrind with this patch
and it looks good.

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 387712] s390x cgijnl reports Conditional jump or move depends on uninitialised value(s)

2017-12-12 Thread Julian Seward
https://bugs.kde.org/show_bug.cgi?id=387712

--- Comment #3 from Julian Seward  ---
Created attachment 109333
  --> https://bugs.kde.org/attachment.cgi?id=109333&action=edit
Proposed fix

This fixes it for me.  Mark, can you give it a spin?

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 387712] s390x cgijnl reports Conditional jump or move depends on uninitialised value(s)

2017-12-08 Thread Mark Wielaard
https://bugs.kde.org/show_bug.cgi?id=387712

--- Comment #2 from Mark Wielaard  ---
 SB 5392 (evchecks 243347) [tid 1] 0x4066170 __libdwfl_frame_unwind+64
/home/mjw/elfutils/libdw/libdw.so+0x35170

 Front end 

cgijnl   %r1,0,.+344
  -- IMark(0x4066170, 6, 0) --
  t0 = GET:I64(200)
  t2 = 0x0:I64
  t1 =
s390_calculate_cond[mcx=0x13]{0x8000fe588}(0xA:I64,0x1:I64,t0,t2,0x0:I64):I32
  if (CmpNE32(t1,0x0:I32)) { PUT(336) = 0x40662C8:I64; exit-Boring
} 
  PUT(336) = 0x4066176:I64
  PUT(336) = GET:I64(336); exit-Boring

GuestBytes 4066170 6  EC 1A 00 AC 00 7C  1EEC


  After tree-building 

IRSB {
   t0:I64   t1:I32   t2:I64   t3:I32   t4:I1   t5:I64   t6:I32   t7:I1


   -- IMark(0x4066170, 6, 0) --
   if (CmpLE64S(0x0:I64,GET:I64(200))) { PUT(336) = 0x40662C8:I64; exit-Boring
} 
   PUT(336) = 0x4066176:I64; exit-Boring
}

VexExpansionRatio 6 88   146 :10

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 387712] s390x cgijnl reports Conditional jump or move depends on uninitialised value(s)

2017-12-08 Thread Mark Wielaard
https://bugs.kde.org/show_bug.cgi?id=387712

--- Comment #1 from Mark Wielaard  ---
 SB 4912 (evchecks 237802) [tid 1] 0x406615c __libdwfl_frame_unwind+44
/home/mjw/elfutils/libdw/libdw.so+0x3515c

 Front end 

lg   %r1,16(%r11)
  -- IMark(0x406615C, 6, 0) --
  t1 = 0x10:I64
  t0 = Add64(Add64(t1,GET:I64(280)),0x0:I64)
  PUT(200) = LDbe:I64(t0)
  PUT(336) = 0x4066162:I64

lg   %r3,176(%r15)
  -- IMark(0x4066162, 6, 0) --
  t3 = 0xB0:I64
  t2 = Add64(Add64(t3,GET:I64(312)),0x0:I64)
  PUT(216) = LDbe:I64(t2)
  PUT(336) = 0x4066168:I64

tmhh %r1,16384
  -- IMark(0x4066168, 4, 0) --
  t4 = GET:I16(200)
  t5 = 0x4000:I16
  PUT(352) = 0x13:I64
  PUT(360) = 16Uto64(t4)
  PUT(368) = 16Uto64(t5)
  PUT(376) = 0x0:I64
  PUT(336) = 0x406616C:I64

jne  .+10
  -- IMark(0x406616C, 4, 0) --
  t6 =
s390_calculate_cond[mcx=0x13]{0x8000fe588}(0x7:I64,GET:I64(352),GET:I64(360),GET:I64(368),GET:I64(376)):I32
  if (CmpNE32(t6,0x0:I32)) { PUT(336) = 0x4066176:I64; exit-Boring
} 
  PUT(336) = 0x4066170:I64
  PUT(336) = GET:I64(336); exit-Boring

GuestBytes 406615C 20  E3 10 B0 10 00 04 E3 30 F0 B0 00 04 A7 12 40 00 A7 74 00
05  06337FED

  After tree-building 

IRSB {
   t0:I64   t1:I64   t2:I64   t3:I64   t4:I16   t5:I16   t6:I32   t7:I64
   t8:I64   t9:I64   t10:I64   t11:I64   t12:I64   t13:I64   t14:I64   t15:I64
   t16:I64   t17:I64   t18:I64   t19:I64   t20:I64   t21:I32   t22:I1   t23:I64
   t24:I64   t25:I32   t26:I1   t27:I64   

   -- IMark(0x406615C, 6, 0) --
   PUT(200) = LDbe:I64(Add64(0x10:I64,GET:I64(280)))
   PUT(336) = 0x4066162:I64
   -- IMark(0x4066162, 6, 0) --
   PUT(216) = LDbe:I64(Add64(0xB0:I64,GET:I64(312)))
   -- IMark(0x4066168, 4, 0) --
   PUT(352) = 0x13:I64
   t24 = 16Uto64(GET:I16(200))
   PUT(360) = t24
   PUT(368) = 0x4000:I64
   PUT(376) = 0x0:I64
   PUT(336) = 0x406616C:I64
   -- IMark(0x406616C, 4, 0) --
   if (CmpNE64(And64(t24,0x4000:I64),0x0:I64)) { PUT(336) = 0x4066176:I64;
exit-Boring } 
   PUT(336) = 0x4066170:I64; exit-Boring
}

VexExpansionRatio 20 168   84 :10

-- 
You are receiving this mail because:
You are watching all bug changes.