[cfe-commits] [PATCH] Fix test case failure due to C++ ABI difference on ARM

Eli Friedman eli.friedman at gmail.com
Mon Apr 30 10:39:41 PDT 2012


On Mon, Apr 30, 2012 at 8:14 AM, 陳韋任 <chenwj at iis.sinica.edu.tw> wrote:
> Hi all,
>
>  In section 3.1 "Summary of differences from and additions to the generic C++
> ABI" [1], it says ARM requires constructor return *this* instead of void. This
> patch fix test cases failures due to this ABI difference on ARM.
>
>  As for CodeGenCXX/x86-64-abi-sret-vs-2word-struct-param.cpp, this is a
> x86-specific test case. Adding triple seems to be a right solution.

Looks fine.

-Eli




More information about the cfe-commits mailing list