[PATCH] D60349: [COFF, ARM64] Fix ABI implementation of struct returns

ssijaric added inline comments. Comment at: lib/CodeGen/MicrosoftCXXABI.cpp:1058 + + // 1. For return types <= 16 bytes, use the C return semantics. + richard.townsend.arm wrote: > Microsoft have updated the spec since this was written, it now says to check > f

[PATCH] D56463: [SEH] Pass the frame pointer from SEH finally to finally functions

This revision was automatically updated to reflect the committed changes. Closed by commit rC351302: [SEH] Pass the frame pointer from SEH finally to finally functions (authored by ssijaric, committed by ). Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56463/new/

[PATCH] D56463: [SEH] Pass the frame pointer from SEH finally to finally functions

ssijaric updated this revision to Diff 181337. ssijaric added a comment. Address formatting comments. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56463/new/ https://reviews.llvm.org/D56463 Files: lib/CodeGen/CGException.cpp test/CodeGen/exceptions-seh-nest

[PATCH] D56463: [SEH] Pass the frame pointer from SEH finally to finally functions

ssijaric created this revision. ssijaric added reviewers: rnk, efriedma, mstorsjo, TomTan. The following test case, compiled with -OO -target=x86_64-windows-win32, returns an incorrect value. It returns 5, when it should return 9. The problem is that the frame pointer that the first finally bl

[PATCH] D49770: [ARM64] [Windows] Follow MS X86_64 C++ ABI when passing structs

This revision was automatically updated to reflect the committed changes. Closed by commit rC338076: [ARM64] [Windows] Follow MS X86_64 C++ ABI when passing structs (authored by ssijaric, committed by ). Repository: rC Clang https://reviews.llvm.org/D49770 Files: include/clang/Basic/TargetI

[PATCH] D49770: [ARM64] [Windows] Follow MS X86_64 C++ ABI when passing structs

ssijaric updated this revision to Diff 157559. ssijaric added a comment. Last change before committing to address Reid's point of using CCK_MicrosoftWin64 for Windows on both X86_64 and ARM64. Repository: rC Clang https://reviews.llvm.org/D49770 Files: include/clang/Basic/TargetInfo.h l

[PATCH] D49770: [ARM64] [Windows] Follow MS X86_64 C++ ABI when passing structs

ssijaric updated this revision to Diff 157394. ssijaric added a comment. Updated to address Martin's observation. Verified that small structs with default copy constructors and non-trivial destructors are passed in registers on ARM64 Windows. Repository: rC Clang https://reviews.llvm.org/D

[PATCH] D49770: [ARM64] [Windows] Follow MS X86_64 C++ ABI when passing structs

ssijaric added a comment. Yes, it looks like the comment for the thumb case is applicable here as well. VC++ on ARM64 passes small structs with a destructor and a default copy constructor in a register, while clang passes them on the stack. I will probably address this in a separate patch.

[PATCH] D49770: [ARM64] [Windows] Follow MS X86_64 C++ ABI when passing structs

ssijaric created this revision. ssijaric added reviewers: rnk, mstorsjo, TomTan, haripul. Herald added a reviewer: javed.absar. Herald added subscribers: cfe-commits, chrib, kristof.beyls. Microsoft's C++ object model for ARM64 is the same as that for X86_64. For example, small structs with non-