On Tue, Jul 24, 2012 at 04:25:51PM -0700, Roland McGrath wrote: > If the point of cpp-various is just to generate testdata, then just compile > it once and make a binary test file to run readelf on.
The point was more to have something like a self-test. If there is a C++ compiler on the system, we want to make sure we recognize the produced DWARF. The current testcase is very simple, it can certainly be expanded to test some "deeper" C++ constructs. > IMHO this should be several separate changes that we review individually. Yeah, it turned out larger than I thought because while writing the test cases I found some stuff that I then added immediately on the branch. And it was late, so I wanted to at least push it out. I'll split them. > I don't really think we need to keep DW_LANG_Objc around. > If there are any users they can update their spelling, but > there probably aren't any. That would be convenient, since then we don't have duplicate names. I was a little paranoid wanting backwards compatibility, but in this case it might be there are no actual users. > Actually, it should be one change to add DW_OP_GNU_parameter_ref to dwarf.h > and handle it in readelf. That's separate from other dwarf.h changes. Yeah, I'll split that one out, it rolled out of adding the testcase. Thanks, Mark _______________________________________________ elfutils-devel mailing list [email protected] https://fedorahosted.org/mailman/listinfo/elfutils-devel
