Andy Gibbs
2012-Jul-23 18:46 UTC
[LLVMdev] Fw: Fix for forward-declare-class.cpp in debuginfo-tests
Hi, Please could someone kindly review this patch for me? Many thanks Andy On Wednesday, July 18, 2012 9:15 AM, Andy Gibbs wrote:> Hi, > > I'm not sure where to post this since it relates to the "debuginfo-tests" > repository, so please tell me where to forward it if this is the wrong > place! > > I've fixed the forward-declare-class.cpp test-case in the attached patch. > > It seems that gdb has fixed their bug in version 7.4, so the patch uses a > regex to match expected output for both the broken and fixed gdb. > > Cheers > Andy-------------- next part -------------- A non-text attachment was scrubbed... Name: fix.diff Type: application/octet-stream Size: 624 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20120723/ddb2b0f9/attachment.obj>
Eric Christopher
2012-Jul-23 19:55 UTC
[LLVMdev] Fix for forward-declare-class.cpp in debuginfo-tests
Applied here: http://llvm.org/viewvc/llvm-project?rev=160639&view=rev Thanks! -eric On Jul 23, 2012, at 11:46 AM, Andy Gibbs <andyg1001 at hotmail.co.uk> wrote:> Hi, > > Please could someone kindly review this patch for me? > > Many thanks > Andy > > > On Wednesday, July 18, 2012 9:15 AM, Andy Gibbs wrote: >> Hi, >> I'm not sure where to post this since it relates to the "debuginfo-tests" >> repository, so please tell me where to forward it if this is the wrong >> place! >> I've fixed the forward-declare-class.cpp test-case in the attached patch. >> It seems that gdb has fixed their bug in version 7.4, so the patch uses a >> regex to match expected output for both the broken and fixed gdb. >> Cheers >> Andy > <fix.diff>_______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev