Barring serious extenuating circumstances, an untested functional change in llvm is not complete. And as time goes on it becomes less and less likely that we'll revisit this. I think it's a good idea to revert this until there's a test.
vedant > On Feb 6, 2018, at 7:42 AM, Davide Italiano via lldb-commits > <[email protected]> wrote: > > ping. I might consider reverting this and the other commits without > tests bay the end of the week unless we have a story for testing here. > It's been one month. > > On Mon, Feb 5, 2018 at 9:44 AM, Davide Italiano <[email protected]> wrote: >> On Mon, Jan 29, 2018 at 10:17 AM, Jason Molenda <[email protected]> wrote: >>> >>> >>>> On Jan 28, 2018, at 9:59 PM, Davide Italiano <[email protected]> wrote: >>>> >>>> On Thu, Jan 11, 2018 at 5:26 PM, Davide Italiano <[email protected]> >>>> wrote: >>>>> On Thu, Jan 11, 2018 at 5:16 PM, Jason Molenda via lldb-commits >>>>> <[email protected]> wrote: >>>>>> Author: jmolenda >>>>>> Date: Thu Jan 11 17:16:13 2018 >>>>>> New Revision: 322339 >>>>>> >>>>>> URL: http://llvm.org/viewvc/llvm-project?rev=322339&view=rev >>>>>> Log: >>>>>> When parsing the target.xml register file, if no architecture has >>>>>> been specified yet (either by the user, or by one of the lldb >>>>>> extensions like qHostInfo or qProcessInfo), and the target.xml >>>>>> includes a <architecture> tag specifying x86_64, set the architecture >>>>>> appropriately. >>>>>> >>>>>> I'm not sure what we can expect to see in the <architecture> tag, so >>>>>> I'm only doing this for x86_64 right now where I've seen "i386:x86_64" >>>>>> used. I've seen a target.xml from a jtag board that sends just "arm" >>>>>> because it doesn't know more specifically what type of board it is >>>>>> connected to... >>>>>> >>>>>> <rdar://problem/29908970> >>>>>> >>>>> >>>>> Jason, >>>>> is there a way to test this change? >>>>> >>>> >>>> ping? Any news on this front? >>> >>> Thanks for the reminder. I need to look at doing this - I'm in the middle >>> of another patch right now but I'll return to this after that. >> >> weekly ping :) >> >> -- >> Davide > _______________________________________________ > lldb-commits mailing list > [email protected] > http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits _______________________________________________ lldb-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
