[llvm-commits] [llvm] r134893 - in /llvm/trunk: include/llvm/DerivedTypes.h include/llvm/Target/TargetData.h lib/CodeGen/SjLjEHPrepare.cpp lib/Target/ARM/ARMGlobalMerge.cpp lib/Target/TargetData.cpp lib/Transforms/IPO/DeadArgumentElimination.cpp

Tobias Grosser tobias at grosser.es
Tue Jul 12 04:42:27 PDT 2011


On 07/11/2011 01:24 PM, Jay Foad wrote:
> On 11 July 2011 12:14, Tobias Grosser<tobias at grosser.es>  wrote:
>> On 07/11/2011 11:56 AM, Jay Foad wrote:
>>> Author: foad
>>> Date: Mon Jul 11 04:56:20 2011
>>> New Revision: 134893
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=134893&view=rev
>>> Log:
>>> De-constify Types in StructType::get() and TargetData::getIntPtrType().
>>
>> Do Builder.getInt8PtrTy() and Co also need the const removed?
>
> I think the plan is to remove the const *everywhere*, eventually. I'm
> just doing the bits that I have some motivation for.

OK. Thanks.

I will deconstify stuff that is obvious and seems useful, as long as I 
believe it will not cause regressions at other places.

Tobi



More information about the llvm-commits mailing list