[llvm-commits] [llvm] r46216 - /llvm/trunk/lib/VMCore/ParameterAttributes.cpp

[Chris Lattner]

On Jan 21, 2008, at 1:37 PM, Duncan Sands wrote:
> URL: http://llvm.org/viewvc/llvm-project?rev=46216&view=rev
> Log:
> It turns out that in C++ it is legal to declare functions
> that return an opaque type by value, as long as you don't
> call it or provide a body (you can take the address of it).
> So it is wrong to insist that sret parameters not be an
> opaque*.  And I guess it is really up to codegen to complain
> if someone tries to call such a function.  I'm also removing
> the analogous check from byval parameters, since I don't
> see why we shouldn't allow them as long as no-one tries to
> call the function or give it a body.

Makes sense, thanks Duncan,

-Chris