[llvm-commits] [PATCH] Fix PR11004: Crash in llc arm backend

Cameron Zwarich zwarich at apple.com
Sun Sep 25 03:46:48 PDT 2011


The fact that the memcpy is not always inlined is an explicit design decision of the ARM byval implementation. Think of the case of copying a gigantic struct.

The nested legalization of calls it required is unfortunate and has bugs, but I believe Dan Gohman has been cleaning it up a bit.

Unless I am just really out of the loop (which is possible) this is not the right fix.

Sent from my iPhone

On Sep 25, 2011, at 1:34 AM, David Meyer <pdox at google.com> wrote:

> Fixes http://llvm.org/bugs/show_bug.cgi?id=11004
> 
> New test included. Seeking approval to commit.
> 
> Thanks,
> pdox
> <arm_byval.patch>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list