<div dir="ltr">Hi Alex,<div><br></div><div>I think my only concern here is that this affects forward compatibility - I don't think there's anything to be done about it, but it does mean that existing installed libraries will fail to read even the most basic of newer modules with this change and worth calling out explicitly.</div><div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Nov 21, 2020 at 8:02 AM Alexander Richardson <<a href="mailto:Alexander.Richardson@cl.cam.ac.uk" target="_blank">Alexander.Richardson@cl.cam.ac.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello all,<br>
<br>
I committed <a href="https://reviews.llvm.org/D84345" rel="noreferrer" target="_blank">https://reviews.llvm.org/D84345</a> yesterday, which adds a<br>
new "-G<N>" field to the DataLayout and specifies the address space<br>
that LLVM passes should use when creating new global variables if<br>
there is no additional contextual information.<br>
<br>
This change is required to improve support for targets that don't use<br>
address space zero for globals (e.g. the CHERI RISC-V/MIPS targets and<br>
Arm Morello) and is required to remove uses of<br>
PointerType::getUnqual() and similar functions<br>
(<a href="http://lists.llvm.org/pipermail/llvm-dev/2020-July/143638.html" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/llvm-dev/2020-July/143638.html</a>).<br>
<br>
This change should be fully backwards compatible: if your target<br>
doesn't specify "-G" in the datalayout string all globals will be<br>
created in AS0 as usual.<br>
Please let me know if you have any concerns about this change.<br>
<br>
Thanks,<br>
Alex<br>
</blockquote></div>