[LLVMdev] [cfe-dev] RFC: Removing ABIArgInfo::Expand from Clang ABI lowering

Tim Northover t.p.northover at gmail.com
Thu Feb 5 19:38:38 PST 2015


Hi John,

On 3 February 2015 at 21:40, John McCall <rjmccall at apple.com> wrote:
> I’ve looked at our currently-supported in-tree uses of Expand, and I feel confident
> that they can all be easily switched to Direct.  However, I am willing to maintain
> Expand if anyone is maintaining an out-of-tree target and cannot simply switch to
> Direct, or if anyone has some other compelling reason Expand needs to stay.
> Otherwise, I’ll be removing it relatively soon.

I think removing it is a good idea too (assuming no OOT user has a
compelling scenario).

By default a struct coercion gets expanded to something equivalent
before lib/Target in the backend sees it anyway (modulo possible weird
union behaviour, which would almost certainly be unintended).

Cheers.

Tim.




More information about the llvm-dev mailing list