Thanks Ranjeet, Indeed I was planning to update the patches to top of trunk and reload them, so if you can save me this trouble I would appreciate that. Once we upload the updated patches I would like to ping the community and ask for feedback.
Thanks, Amjad > -----Original Message----- > From: Ranjeet Singh [mailto:ranjeet.si...@arm.com] > Sent: Friday, July 22, 2016 20:16 > To: Aboud, Amjad <amjad.ab...@intel.com>; rich...@metafoo.co.uk; > paul.robin...@sony.com; apra...@apple.com > Cc: ranjeet.si...@arm.com; cfe-commits@lists.llvm.org > Subject: Re: [PATCH] D16135: Macro Debug Info support in Clang > > rs added a subscriber: rs. > rs added a comment. > > Hi Amjad, > > are you still planning on getting this patch and > https://reviews.llvm.org/D16077 committed ? It looks like these two patches > are final pieces in the puzzle to get macro information in the DWARF debug > output. > > I've downloaded the diffs and applied them myself on my local checkout and > they seem to work fine. If you would like me to upload the rebased patches > onto phabricator to save you the trouble of having to the fix conflicts > downstream then let me know. > > Thanks, > Ranjeet > > > https://reviews.llvm.org/D16135 > > --------------------------------------------------------------------- Intel Israel (74) Limited This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
