[RFA] create include/cgen
Doug Evans
dje@sebabeach.org
Tue Oct 20 00:15:00 GMT 2009
Hans-Peter Nilsson wrote:
> On Mon, 19 Oct 2009, Doug Evans wrote:
>
>> This patch creates include/cgen and moves a few files there.
>>
>
> Should the cgen CVS module now include naked-include
> or at least src/include/cgen?
>
At this point in time I'd tend to think not.
The cgen cvs module currently doesn't compile any code.
>
>> It also simplifies basic-modes.h (old cgen-types.h) and
>> basic-ops.h (old cgen-ops.h) by removing floating point support.
>>
>
> Do the same in sim/common as here regarding cgen-ops.h?
>
> Anyway, *please* merge in the changes from
> src/sim/common/cgen-ops.h into cgen/basic-ops.h; see
> <http://sourceware.org/ml/cgen/2008-q4/msg00020.html>.
>
> brgds, H-P
>
Yeah, my wish/plan is to have one copy, in include/cgen.
There's still an issue of having multiple copies of code
(cgen-ops.h/basic-ops.h declare some functions, and will declare more
over time, and we'd ideally have only copy).
I don't mind having it all live in libopcodes (at least for now, and
ports that don't use cgen won't see it) instead of creating libcgen.
[Adding another library that binutils binaries have to link is
problematic. It's doable, but it's work and I can well imagine folks
not wanting it. That's fine. There's already libopcodes which "works
for me".]
Thanks for the URL. No worries, I'll make sure to merge them.
More information about the Binutils
mailing list