diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2014-02-25 22:23:04 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2014-02-25 22:23:04 +0000 |
commit | 3f0a9af13bec64f3bc4bee06a57e18feeb766570 (patch) | |
tree | 9de885f1be5c4621a2c98e276f3f2e62324947e8 /unittests | |
parent | 9c256eccb355fb26072ad5353a2f886f62118aa8 (diff) | |
download | llvm-3f0a9af13bec64f3bc4bee06a57e18feeb766570.tar.gz llvm-3f0a9af13bec64f3bc4bee06a57e18feeb766570.tar.bz2 llvm-3f0a9af13bec64f3bc4bee06a57e18feeb766570.tar.xz |
Fix resetting the DataLayout in a Module.
No tool does this currently, but as everything else in a module we should be
able to change its DataLayout.
Most of the fix is in DataLayout to make sure it can be reset properly.
The test uses Module::setDataLayout since the fact that we mutate a DataLayout
is an implementation detail. The module could hold a OwningPtr<DataLayout> and
the DataLayout itself could be immutable.
Thanks to Philip Reames for pushing me in the right direction.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@202198 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'unittests')
-rw-r--r-- | unittests/IR/IRBuilderTest.cpp | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/unittests/IR/IRBuilderTest.cpp b/unittests/IR/IRBuilderTest.cpp index a0a72e82ed..e6db8c66df 100644 --- a/unittests/IR/IRBuilderTest.cpp +++ b/unittests/IR/IRBuilderTest.cpp @@ -108,6 +108,14 @@ TEST_F(IRBuilderTest, LandingPadName) { EXPECT_EQ(LP->getName(), "LP"); } +TEST_F(IRBuilderTest, DataLayout) { + OwningPtr<Module> M(new Module("test", Ctx)); + M->setDataLayout("e-n32"); + EXPECT_TRUE(M->getDataLayout()->isLegalInteger(32)); + M->setDataLayout("e"); + EXPECT_FALSE(M->getDataLayout()->isLegalInteger(32)); +} + TEST_F(IRBuilderTest, GetIntTy) { IRBuilder<> Builder(BB); IntegerType *Ty1 = Builder.getInt1Ty(); |