[PATCH] D13683: Remove File from createSubroutineType

Duncan P. N. Exon Smith via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 12 20:07:42 PDT 2015


> On 2015-Oct-12, at 17:56, Amaury SECHET <deadalnix+llvmreview at gmail.com> wrote:
> 
> deadalnix created this revision.
> deadalnix added reviewers: chandlerc, echristo, dexonsmith.
> deadalnix added a subscriber: llvm-commits.
> Herald added a subscriber: axw.
> 
> It is not used. I kept the old functiona round for backward compatibility, so we have a window to migrate code.
> 
> http://reviews.llvm.org/D13683
> 
> Files:
>  bindings/go/llvm/DIBuilderBindings.cpp
>  include/llvm/IR/DIBuilder.h
>  lib/IR/DIBuilder.cpp
>  unittests/IR/IRBuilderTest.cpp
>  unittests/Transforms/Utils/Cloning.cpp
> 
> <D13683.37205.patch>

LGTM.  Thanks for the cleanup.


More information about the llvm-commits mailing list