[PATCH] Microblaze ABI and function attributes

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] Microblaze ABI and function attributes

Wesley Peck-2
Attached are two patches that must be applied together but are separated for clarity. The first patch adds two attributes, MBlazeInterruptHandler and MBlazeSaveVolatiles, that map to the Microblaze function attributes "interrupt_handler" and "save_volatiles". The second patch adds some ABI information for the Microblaze to lib/CodeGen/TargetInfo.cpp and lowers the two attributes from the first patch to the correct LLVM calling conventions. 

These patches pair with the LLVM commits 121888 and 121891 to lower the attributes into the correct machine code. Please let me know if it is OK to commit these changes.

Thanks,
--
Wesley Peck
University of Kansas
SLDG Laboratory




_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev

mblaze-abi.patch (4K) Download Attachment
mblaze-attrs.patch (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] [PING] Microblaze ABI and function attributes

Wesley Peck-2
Ping!
--
Wesley Peck
University of Kansas
SLDG Laboratory




On Dec 15, 2010, at 2:55 PM, Wesley Peck wrote:

Attached are two patches that must be applied together but are separated for clarity. The first patch adds two attributes, MBlazeInterruptHandler and MBlazeSaveVolatiles, that map to the Microblaze function attributes "interrupt_handler" and "save_volatiles". The second patch adds some ABI information for the Microblaze to lib/CodeGen/TargetInfo.cpp and lowers the two attributes from the first patch to the correct LLVM calling conventions. 

These patches pair with the LLVM commits 121888 and 121891 to lower the attributes into the correct machine code. Please let me know if it is OK to commit these changes.

Thanks,
--
Wesley Peck
University of Kansas
SLDG Laboratory

_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev

mblaze-abi.patch (4K) Download Attachment
mblaze-attrs.patch (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] [PING] Microblaze ABI and function attributes

Dale Johannesen
This is MBlaze specific, so I'm not sure who could review it.  I can say it looks analogous to other target code and is unlikely to break any other target.

On Dec 17, 2010, at 6:30 AM, Wesley Peck wrote:

Ping!
--
Wesley Peck
University of Kansas
SLDG Laboratory

<mblaze-abi.patch>
<mblaze-attrs.patch>

On Dec 15, 2010, at 2:55 PM, Wesley Peck wrote:

Attached are two patches that must be applied together but are separated for clarity. The first patch adds two attributes, MBlazeInterruptHandler and MBlazeSaveVolatiles, that map to the Microblaze function attributes "interrupt_handler" and "save_volatiles". The second patch adds some ABI information for the Microblaze to lib/CodeGen/TargetInfo.cpp and lowers the two attributes from the first patch to the correct LLVM calling conventions. 

These patches pair with the LLVM commits 121888 and 121891 to lower the attributes into the correct machine code. Please let me know if it is OK to commit these changes.

Thanks,
--
Wesley Peck
University of Kansas
SLDG Laboratory
_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev


_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] [PING] Microblaze ABI and function attributes

Chris Lattner
Go ahead and commit it, feel free to maintain mblaze specific code as you see fit,

-Chris

On Dec 17, 2010, at 2:48 PM, dalej wrote:

This is MBlaze specific, so I'm not sure who could review it.  I can say it looks analogous to other target code and is unlikely to break any other target.

On Dec 17, 2010, at 6:30 AM, Wesley Peck wrote:

Ping!
--
Wesley Peck
University of Kansas
SLDG Laboratory

<mblaze-abi.patch>
<mblaze-attrs.patch>

On Dec 15, 2010, at 2:55 PM, Wesley Peck wrote:

Attached are two patches that must be applied together but are separated for clarity. The first patch adds two attributes, MBlazeInterruptHandler and MBlazeSaveVolatiles, that map to the Microblaze function attributes "interrupt_handler" and "save_volatiles". The second patch adds some ABI information for the Microblaze to lib/CodeGen/TargetInfo.cpp and lowers the two attributes from the first patch to the correct LLVM calling conventions. 

These patches pair with the LLVM commits 121888 and 121891 to lower the attributes into the correct machine code. Please let me know if it is OK to commit these changes.

Thanks,
--
Wesley Peck
University of Kansas
SLDG Laboratory
_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev

_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev


_______________________________________________
cfe-dev mailing list
[hidden email]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev