[LLVMdev] RFC: llvm-convert.cpp Patch
isanbard at gmail.com
Wed Nov 7 10:58:53 PST 2007
On 11/7/07, Duncan Sands <baldrick at free.fr> wrote:
> > How about this patch then?
> How about this one :)
Your patch didn't apply cleanly, so I couldn't test it out. :-(
> It passes alignment and volatility around
> with DestLoc. It's not finished because I noticed some bugs in
> how CopyAggregate and ZeroAggregate handle alignment (problem points
> marked with "QQ"). Also, I noticed potential problems with how we
> handle call arguments and return results (what if they are strangely
> aligned/volatile?), marked with FIXME.
> While there, I fixed the following:
> - Taught EmitAggregateZero not to do element by element zeroing
> if the LLVM type doesn't cover the gcc type. Also, not to do
> it if there are too many aggregate elements.
> - Added a bunch of missing alignment/volatile stuff on various
> loads and stores.
> - Fixed up a lot of bogus handling of volatility in the complex
> number stuff.
More information about the llvm-dev