[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