Re: [llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-31 Thread Duncan Sands
Hi Dale, maybe you could use IntrinsicInst::StripPointerCasts instead? Didn't know about this, thanks. It recurses and I don't think that's what I want in this case. I did, however, steal its looping code:) Are you sure you don't want it to be recursive? I can take a global variable,

Re: [llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-31 Thread Dale Johannesen
On Jan 31, 2008, at 9:55 AM, Duncan Sands wrote: Hi Dale, maybe you could use IntrinsicInst::StripPointerCasts instead? Didn't know about this, thanks. It recurses and I don't think that's what I want in this case. I did, however, steal its looping code:) Are you sure you don't want

Re: [llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-31 Thread Chris Lattner
On Jan 31, 2008, at 10:46 AM, Dale Johannesen wrote: No I'm not sure, but I know of no case where this is useful or desirable. The code in question is handling the debug info for global variables, specifically the part that points to the global variable itself. That's under the control of

Re: [llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-31 Thread Duncan Sands
Hi Dale, maybe you could use IntrinsicInst::StripPointerCasts instead? Didn't know about this, thanks. It recurses and I don't think that's what I want in this case. I did, however, steal its looping code:) Are you sure you don't want it to be recursive? I can take a global

[llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-30 Thread Dale Johannesen
Author: johannes Date: Wed Jan 30 13:00:21 2008 New Revision: 46565 URL: http://llvm.org/viewvc/llvm-project?rev=46565view=rev Log: Accept getelementptr starting at GV with all 0 indices as a legitimate way of representing global variable GV in debug info. Added:

Re: [llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-30 Thread Chris Lattner
On Jan 30, 2008, at 11:00 AM, Dale Johannesen wrote: URL: http://llvm.org/viewvc/llvm-project?rev=46565view=rev Log: Accept getelementptr starting at GV with all 0 indices as a legitimate way of representing global variable GV in debug info. Thanks Dale, A minor simplification: +++

Re: [llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-30 Thread Duncan Sands
Hi Dale, maybe you could use IntrinsicInst::StripPointerCasts instead? Best wishes, Duncan. ___ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Re: [llvm-commits] [llvm] r46565 - in /llvm/trunk: lib/CodeGen/MachineModuleInfo.cpp test/DebugInfo/globalGetElementPtr.ll

2008-01-30 Thread Dale Johannesen
On Jan 30, 2008, at 11:10 AM, Duncan Sands wrote: Hi Dale, maybe you could use IntrinsicInst::StripPointerCasts instead? Best wishes, Duncan. Didn't know about this, thanks. It recurses and I don't think that's what I want in this case. I did, however, steal its looping code:)