[PATCH] D64931: Change X86 datalayout for three address spaces that specify pointer sizes.
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 8 11:07:26 PDT 2019
lebedev.ri added inline comments.
================
Comment at: clang/lib/Basic/Targets/OSTargets.h:771-772
} else if (Triple.getArch() == llvm::Triple::x86) {
- this->resetDataLayout("e-m:e-p:32:32-i64:64-n8:16:32-S128");
+ this->resetDataLayout("e-m:e-p:32:32-p253:32:32-p254:32:32-p255:64:64-"
+ "i64:64-n8:16:32-S128");
} else if (Triple.getArch() == llvm::Triple::x86_64) {
----------------
I'd expect that this should be guarded by whatever flag is used for ms extensions.
Put differently, i i'm not sure that when those extensions are not enabled, the datalayout should be changed?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64931/new/
https://reviews.llvm.org/D64931
More information about the cfe-commits
mailing list