[PATCH] D66843: Change datalayout compatibility check for X86 to allow datalayouts without the new address spaces.

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 13 11:46:47 PDT 2019


No, it's exactly that. Though there are possibly other ways to handle it if
there are problems.

On Fri, Sep 13, 2019, 11:43 AM Amy Huang via Phabricator <
reviews at reviews.llvm.org> wrote:

> akhuang added a comment.
>
> > I really do think you should just upgrade old layouts to include the
> address space.
>
> Maybe I'm misunderstanding--is this different from the AutoUpgrade thing
> that @rnk suggested?
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D66843/new/
>
> https://reviews.llvm.org/D66843
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190913/5c9f6115/attachment.html>


More information about the llvm-commits mailing list