[llvm-commits] [llvm] r102831 - in /llvm/trunk: lib/Transforms/IPO/Inliner.cpp test/Transforms/Inline/delete-call.ll

[Duncan Sands]

>> Disable the call-deletion transformation introduced in r86975.  Without
>> halting analysis, it is illegal to delete a call to a read-only function.
>> The correct solution is almost certainly to add a "must halt" attribute and
>> only allow deletions in its presence.
>>
>> XFAIL the relevant testcase for now.
>
> This is not he right approach to solving this, so I reverted it.  If you'd like to talk about this, lets do.  There are several related open issues, please don't run around reverting random patches.
>

This is PR965 by the way.

Ciao,

Duncan.