[PATCH] D70947: Add a default address space for globals to DataLayout

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

Needs some bitcode compatibility tests, and actual uses. AMDGPU's data layout should also add -G1



================
Comment at: llvm/docs/LangRef.rst:2059
+``G<address space>``
+    Specifies the address space that should be used for global variables.
+    If omitted, the globals address space defaults to the default address
----------------
The phrasing is a bit wrong. This implies for all global variables. This should maybe be the default address space for globals?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70947/new/

https://reviews.llvm.org/D70947