[LLVMbugs] [Bug 5058] Support for sret the msvc++ way.

bugzilla-daemon at llvm.org bugzilla-daemon at llvm.org
Mon Mar 12 18:06:43 PDT 2012


http://llvm.org/bugs/show_bug.cgi?id=5058

Aaron Ballman <aaron at aaronballman.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |WORKSFORME

--- Comment #3 from Aaron Ballman <aaron at aaronballman.com> 2012-03-12 20:06:43 CDT ---
Sorry, I wasn't clear.  I believe it is fixed because all of the test cases
(thiscall-struct-return.cpp, thiscall-struct-return.ll) claim it is so, and my
own testing shows it to be so.  I said "believe" simply because I've learned
that my testing != other's testing and so I'm more cautious about assertions.

r151122, r151123 are both sret related (with thiscall) and AFAIK were the last
outstanding issue.

Feel free to test for yourself to verify, however.  I verified: 4, 8, 16, 3, 7,
15 byte structures with packing sizes 1, 2, 4, 8, and 16 with cdecl, stdcall,
fastcall and thiscall.

-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.



More information about the llvm-bugs mailing list