[PATCH] D19469: [TLI] Unify LibFunc signature checking. NFCI.
Ahmed Bougacha via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 24 13:38:21 PDT 2016
ab added a comment.
In http://reviews.llvm.org/D19469#410333, @joker.eph wrote:
> > I thought about making TLI require DataLayout as well, but you'd have the same problem of picking a default.
>
>
> I'm not sure to follow: if you require the DataLayout why would you have a problem with a "default"? You always have the value don't you?
The analysis isn't always built with a module context; the 0-argument constructor would only have the empty DataLayout.
http://reviews.llvm.org/D19469
More information about the llvm-commits
mailing list