[Libclc-dev] [PATCH 1/2] amdgcn: Fix datalayout after clang r324101

[Matt Arsenault via Libclc-dev]

> On Feb 9, 2018, at 14:23, Jan Vesely via Libclc-dev <libclc-dev at lists.llvm.org> wrote:
> 
> On Fri, 2018-02-09 at 13:35 -0500, Matt Arsenault wrote:
>>> On Feb 8, 2018, at 15:22, Jan Vesely via Libclc-dev <libclc-dev at lists.llvm.org> wrote:
>>> 
>>> +target datalayout = "e-p:32:32-p1:64:64-p2:64:64-p3:32:32-p4:64:64-p5:32:32-i64:64-v16:16-v24:32-v32:32-v48:64-v96:128-v192:256-v256:256-v512:512-v1024:1024-v2048:2048-n32:64”
>> 
>> 
>> This is also missing the -A5 that was added previously
> 
> What file is that? .39.ll and .40.ll files use the old datalayout.
> This is why it's a good idea to quote the whole patch instead of just
> snippet.
> 
> Jan_______________________________________________
> 

Oh, I got confused and thought this was the update for the reverted patch adding a new address space to the list. There have been two datalayout changes recently. The new default / future mandatory -A5 in the datalayout and the more recent addition of the -p6 address space

-Matt