[llvm-commits] [llvm] r169218 - in /llvm/trunk: include/llvm/ lib/CodeGen/AsmPrinter/ lib/VMCore/ test/CodeGen/ARM/ test/CodeGen/Thumb/ test/CodeGen/X86/ test/DebugInfo/ test/DebugInfo/X86/ test/JitListener/

Bill Wendling isanbard at gmail.com
Wed Dec 5 11:25:58 PST 2012


On Dec 5, 2012, at 11:07 AM, Eric Christopher <echristo at gmail.com> wrote:

> > Awesome. FWIW we're probably getting those cases wrong in debug info :)
> >
> We're definitely getting the negative lower bounds wrong, since the value's retrieved as a uint64_t. The zero case will be omitted for Ada and friends.
> 
> Agreed. If you'd like to fix it that'd be awesome, if not file a bug and I'll get to it eventually.
> 
Okay. The fix is to emit the lower bound in all cases. I don't think this will be a problem for the debugger. It just adds extra information that it'll probably ignore in C/C++. :)

-bw






More information about the llvm-commits mailing list