[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