[llvm] r210516 - Delete X86JITInfo in the subtarget destructor.

Eric Christopher echristo at gmail.com
Tue Jun 10 11:36:59 PDT 2014


On Tue, Jun 10, 2014 at 8:06 AM, David Blaikie <dblaikie at gmail.com> wrote:
> On Tue, Jun 10, 2014 at 1:03 AM, Eric Christopher <echristo at gmail.com> wrote:
>> Author: echristo
>> Date: Tue Jun 10 03:03:42 2014
>> New Revision: 210516
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=210516&view=rev
>> Log:
>> Delete X86JITInfo in the subtarget destructor.
>>
>> Modified:
>>     llvm/trunk/lib/Target/X86/X86Subtarget.cpp
>>
>> Modified: llvm/trunk/lib/Target/X86/X86Subtarget.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86Subtarget.cpp?rev=210516&r1=210515&r2=210516&view=diff
>> ==============================================================================
>> --- llvm/trunk/lib/Target/X86/X86Subtarget.cpp (original)
>> +++ llvm/trunk/lib/Target/X86/X86Subtarget.cpp Tue Jun 10 03:03:42 2014
>> @@ -363,6 +363,7 @@ X86Subtarget::~X86Subtarget() {
>>    delete TLInfo;
>>    delete InstrInfo;
>>    delete FrameLowering;
>> +  delete JITInfo;
>
> An opportunity for some std::unique_ptrs?
>

Mmm.. probably. :)

-eric

>>  }
>>
>>  bool
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list