[llvm-commits] [PATCH] Add the byval attribute and a bit more

Rafael Espindola espindola at google.com
Thu Jul 5 10:59:47 PDT 2007


The first email bounced, so lets try again.

This patch implements the first part of the proposed solution to bug
1521. It adds the byval attribute and propagates it to DAG level.

The patch also changes the matching code for CCIfType, so that no
backend matches a byval argument. Instead, a backend should be
modified to use a CCIfStruct (see the X86 backend). For now CCIfStruct
is just a placeholder that aborts at run time.

Then plan now is to implement a CCIfStruct that is common to all
backends and just reproduces the same behavior of llvm-gcc. Later on
we can change this to allow for backend specific code to actually fix
the bug.

Sounds like the right direction? OK to commit to trunk?

Cheers,
-- 
Rafael Avila de Espindola

Google Ireland Ltd.
Gordon House
Barrow Street
Dublin 4
Ireland

Registered in Dublin, Ireland
Registration Number: 368047
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm.patch
Type: text/x-patch
Size: 6452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20070705/d56b24b4/attachment.bin>


More information about the llvm-commits mailing list