This is an archive of the discontinued LLVM Phabricator instance.

[Xtensa] Support for variable arguments
Needs RevisionPublic

Authored by andreisfr on Mar 27 2023, 5:35 AM.

Diff Detail

Event Timeline

andreisfr created this revision.Mar 27 2023, 5:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 27 2023, 5:35 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
andreisfr requested review of this revision.Mar 27 2023, 5:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 27 2023, 5:35 AM
JKRhb added a subscriber: JKRhb.Mar 30 2023, 12:57 PM
arsenm requested changes to this revision.Aug 17 2023, 3:22 PM

Needs tests

llvm/lib/Target/Xtensa/XtensaISelLowering.cpp
410

getStoreSize

436

subtarget should be available in the class already

451–453

this is broken, you shouldn't do this

llvm/lib/Target/Xtensa/XtensaMachineFunctionInfo.h
34

Register / MCRegister instead of unsigned?

This revision now requires changes to proceed.Aug 17 2023, 3:22 PM