[llvm-commits] [llvm] r43083 - in /llvm/trunk: lib/Transforms/IPO/RaiseAllocations.cpp test/Transforms/RaiseAllocations/2007-10-17-InvokeFree.ll

[Duncan Sands]

Hi Devang,

> >> Do not raise free() call that is called through invoke instruction.
> 
> > why not?  As far as I know free cannot raise an exception,
> 
> If this is guaranteed then raiseallocs needs to do cleanup and break  
> link between bb with inovke .. at free(..) and corresponding unwind block  
> (remove PHI nodes' incoming edge).

it is not clear to me whether we are allowed to assume that free is the
standard C free (likewise for malloc).  If we are, then I think it would
be better to raise calls to them, fixing up the unwind edge if they were
invokes.

> > and we
> > don't support trapping exceptions for the moment.

Ciao,

Duncan.