[PATCH] Move arch/tdesc.h to common/common-tdesc.h

Yao Qi qiyaoltc@gmail.com
Tue Feb 6 16:28:00 GMT 2018


Alan Hayward <Alan.Hayward@arm.com> writes:

> That results in common/tdesc.h and gdbserver/tdesc.h
> That’s not a problem by itself, because we can just prefix the dir
> name as part of the include.
>

I don't see how does the name of *.c file affect the name of *.h file.
We can still use common/tdesc.h no matter what the *.c name is.

> My next set of patches will then add common/tdesc.c.
> This is slightly more tricky because we now build two tdesc.o files.

I am inclined to create common/ directory in gdb and gdbserver build
tree, and put the object files in the right directory.

> The gdbserver IPA rules will need tweaking, and I’m not sure the OBS
> list will work as it is (given it doesn’t have any dir names).
>
> Like you, I wasn’t sure if there was a naming convention for common/.
> In the end I went with the easiest solution :)

-- 
Yao (齐尧)



More information about the Gdb-patches mailing list