Re: [PATCH v2] sparse: add LLVM code generation backend

[Jeff Garzik <jeff@garzik.org>]

Christopher Li wrote:
> On Mon, Apr 27, 2009 at 5:32 PM, Jeff Garzik <jeff@garzik.org> wrote:
>> Not true -- I can walk SYM_STRUCT of the function arguments' base_type
>> passed to a SYM_FN.  Similarly so for struct-based variable declarations.
>>
>> With that information, you can easily back-reference lvalue uses to the
>> original struct.
> 
> Let say I follow this route, isn't that you can apply the same trick for the
> linearize instruction case? struct instruction has a type member give a
> pointer to C type.
> 
> I still don't see a reason why you have to use your own AST recursive
> code.

You mean, besides the reasons already listed?  Namely, no upstream 
changes are required, and I already have something that works.

Sure, the same trick can be applied.  But that requires a total backend 
rewrite plus dealing with linearize obstacles already described (ref 
linearize_load_gen, linearize_store_gen).  Thus it is obviously a lot 
more work, with additional obstacles (patching upstream, which affects 
existing users), just get back to achieving the same result :)

	Jeff