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

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

In D66843#1669728 <https://reviews.llvm.org/D66843#1669728>, @kristina wrote:

> If this is only being used for `clang-cl` (or `-fms-compatibility`) mode only, is it possible to limit it to just that rather than it affecting pretty much everything?


We decided we'd rather not do that. There needs to be some way to upgrade the x86 datalayout. It can't be frozen in stone forever.

Of course, we need to support the use case of linking old bitcode with new bitcode. Doing that probably requires adding an autoupgrade to replace the old data layout with the new one when loading bitcode. We already have x86-specific intrinsic upgrades in llvm/lib/IR/AutoUpgrade.cpp, so it seems reasonable to add this there, even though it is x86 specific, and remove the X86TargetMachine change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66843