[PATCH] D11021: Make TargetTransformInfo keeping a reference to the Module DataLayoutDataLayout is no longer optional. It was initialized with or withouta DataLayout, and the DataLayout when supplied could have been theone from the TargetMachine.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed Jul 8 05:50:56 PDT 2015


LGTM

On 7 July 2015 at 20:51, Mehdi AMINI <mehdi.amini at apple.com> wrote:
>
> joker.eph added a reviewer: echristo.
> joker.eph added subscribers: yaron.keren, rafael, llvm-commits.
> Herald added a subscriber: jholewinski.
>
> This change is part of a series of commits dedicated to have a single
> DataLayout during compilation by using always the one owned by the
> module.
>
> http://reviews.llvm.org/D11021
>
> Files:
>   include/llvm/Analysis/TargetTransformInfo.h
>   include/llvm/Analysis/TargetTransformInfoImpl.h
>   include/llvm/CodeGen/BasicTTIImpl.h
>   lib/Analysis/TargetTransformInfo.cpp
>   lib/CodeGen/BasicTargetTransformInfo.cpp
>   lib/Target/AArch64/AArch64TargetTransformInfo.h
>   lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
>   lib/Target/AMDGPU/AMDGPUTargetTransformInfo.h
>   lib/Target/ARM/ARMTargetTransformInfo.cpp
>   lib/Target/ARM/ARMTargetTransformInfo.h
>   lib/Target/Mips/MipsTargetMachine.cpp
>   lib/Target/NVPTX/NVPTXTargetMachine.cpp
>   lib/Target/NVPTX/NVPTXTargetTransformInfo.h
>   lib/Target/PowerPC/PPCTargetTransformInfo.h
>   lib/Target/SystemZ/SystemZTargetTransformInfo.h
>   lib/Target/TargetMachine.cpp
>   lib/Target/X86/X86TargetTransformInfo.h
>   lib/Target/XCore/XCoreTargetMachine.cpp
>   lib/Target/XCore/XCoreTargetTransformInfo.h
>



More information about the llvm-commits mailing list