Support call ABI. For now only Linux C and X86_64_SysV calling conventions supported. Variadic function not supported.
Details
Diff Detail
- Build Status
Buildable 9456 Build 9456: arc lint + arc unit
Event Timeline
lib/Target/X86/X86CallLowering.cpp | ||
---|---|---|
324 | Since you do not modify OrigArg you can define it as const & | |
341 | Could you please add a test that checks the case where the callee is a register? | |
348 | typo arguments | |
351 | Please consider having a separate method for handling call results. | |
368 | Could you please document the immediate additions? | |
test/CodeGen/X86/GlobalISel/callingconv.ll | ||
360 | Could you please add tests that exercise arguments and return values that are spitted (like structures)? |
lib/Target/X86/X86CallLowering.cpp | ||
---|---|---|
341 | test_indirect_call cover this case |
lib/Target/X86/X86CallLowering.cpp | ||
---|---|---|
85 | You don't realy need to pass the DataLayout. MachineFunction &MF = MIRBuilder.getMF(); Same goes for STI. | |
92 | Consider calling DL.getPointerSizeOnBits(0) once and assign it to a variable | |
117 | Please don't submit commented code | |
130 | Since you don't handle variadic functions, an assert should be added before returning true |
- Fix according to comments and rebase.
lib/Target/X86/X86CallLowering.cpp | ||
---|---|---|
130 | I need graceful fallback to DAGSEL in case something not supported yet. Eventually it should be supported. |
lib/Target/X86/X86CallLowering.cpp | ||
---|---|---|
140 | I am not sure why you made this members as public but i believe that they should be private as no one outside the class scope should change them. It is especially important for non const members. | |
test/CodeGen/X86/GlobalISel/callingconv.ll | ||
1–3 | Remove --check-prefix=X64_GISEL which is not needed. | |
360 | Where are the tests for returned/passed structures? |
test/CodeGen/X86/GlobalISel/callingconv.ll | ||
---|---|---|
360 | struct not supported yet. |
You don't realy need to pass the DataLayout.
You can get it from the MIRBuilder, something like this:
MachineFunction &MF = MIRBuilder.getMF();
auto &DL = MF.getDataLayout();
Same goes for STI.