[llvm-commits] [llvm] r138636 - /llvm/trunk/lib/Target/ARM/Disassembler/ARMDisassembler.cpp

[Jim Grosbach]

On Aug 26, 2011, at 11:30 AM, Benjamin Kramer wrote:

> On Fri, Aug 26, 2011 at 11:26, Jim Grosbach <grosbach at apple.com> wrote:
>> Testcase?
> 
> It's not directly testable from llvm-mc, but the valgrind buildbot
> complains about uninitialized variables without this patch.

That's odd. Along what execution path? The MC dissassembler and the Enhanced disassembler both early exit when failure is returned and never reference the Size value. At least according to my reading the code, anyway. What am I missing?

-Jim