diff options
author | Rui Ueyama <ruiu@google.com> | 2013-12-20 10:02:59 +0000 |
---|---|---|
committer | Rui Ueyama <ruiu@google.com> | 2013-12-20 10:02:59 +0000 |
commit | 4af032dce7321310d014bf7485e35820e59dbe8f (patch) | |
tree | 0a9377af1c6ff1eb52095210f1b0aad4bc00f8d1 /lld/unittests/DriverTests/WinLinkDriverTest.cpp | |
parent | 556416dab8a6ed9dfedbc33580cc6390af8a143f (diff) | |
download | bcm5719-llvm-4af032dce7321310d014bf7485e35820e59dbe8f.tar.gz bcm5719-llvm-4af032dce7321310d014bf7485e35820e59dbe8f.zip |
[PECOFF] Assign default export ordinals in LinkingContext::verify().
Default ordinals were assigned in EdataPass, and the assigned values were
then discarded in the pass. No code other than EdataPass would not be able
to get all of the information about ordinals. That's not ideal since I'm
writing code to emit an Import Library file, which also needs ordinals.
This is a patch to move the code to assign default ordinals from EdataPass
to LinkingContext::verify(), so that assigned ordinals will be available
anywhere.
No functionality change.
llvm-svn: 197797
Diffstat (limited to 'lld/unittests/DriverTests/WinLinkDriverTest.cpp')
-rw-r--r-- | lld/unittests/DriverTests/WinLinkDriverTest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lld/unittests/DriverTests/WinLinkDriverTest.cpp b/lld/unittests/DriverTests/WinLinkDriverTest.cpp index 2568da8e819..b1a00c30421 100644 --- a/lld/unittests/DriverTests/WinLinkDriverTest.cpp +++ b/lld/unittests/DriverTests/WinLinkDriverTest.cpp @@ -162,7 +162,7 @@ TEST_F(WinLinkParserTest, Export) { _context.getDllExports(); EXPECT_TRUE(exports.size() == 1); EXPECT_EQ("foo", exports[0].name); - EXPECT_EQ(-1, exports[0].ordinal); + EXPECT_EQ(1, exports[0].ordinal); EXPECT_FALSE(exports[0].noname); EXPECT_FALSE(exports[0].isData); } |