diff options
author | Sam Clegg <sbc@chromium.org> | 2023-04-04 10:24:40 -0700 |
---|---|---|
committer | Tom Stellard <tstellar@redhat.com> | 2023-06-10 15:58:16 -0700 |
commit | 7cbf1a2591520c2491aa35339f227775f4d3adf6 (patch) | |
tree | e74f3be630ad2243437e6ddd4ae03d71fdfeee7b | |
parent | 10c3bdc04183cba06fbe7318d1a496fa192f12df (diff) |
[lld][WebAssembly] Fix stub library parsing with windows line endingsllvmorg-16.0.6upstream/release/16.x
Also, fix checking of first line in ::parse. We can't use the
::getLines helper here since that already does comment stripping
internally.
Differential Revision: https://reviews.llvm.org/D147548
(cherry picked from commit d9d840cdaf51a9795930750d1b91d614a3849137)
-rw-r--r-- | lld/test/wasm/Inputs/.gitattributes | 3 | ||||
-rw-r--r-- | lld/wasm/Driver.cpp | 2 | ||||
-rw-r--r-- | lld/wasm/InputFiles.cpp | 18 |
3 files changed, 15 insertions, 8 deletions
diff --git a/lld/test/wasm/Inputs/.gitattributes b/lld/test/wasm/Inputs/.gitattributes new file mode 100644 index 000000000000..6622904075b2 --- /dev/null +++ b/lld/test/wasm/Inputs/.gitattributes @@ -0,0 +1,3 @@ +# ensures that we test parsing of stub libraries that contain +# windows line endings +libstub.so text eol=crlf diff --git a/lld/wasm/Driver.cpp b/lld/wasm/Driver.cpp index 8f7ed8b802d0..762d4c97b099 100644 --- a/lld/wasm/Driver.cpp +++ b/lld/wasm/Driver.cpp @@ -281,7 +281,7 @@ void LinkerDriver::addFile(StringRef path) { files.push_back(createObjectFile(mbref)); break; case file_magic::unknown: - if (mbref.getBuffer().starts_with("#STUB\n")) { + if (mbref.getBuffer().starts_with("#STUB")) { files.push_back(make<StubFile>(mbref)); break; } diff --git a/lld/wasm/InputFiles.cpp b/lld/wasm/InputFiles.cpp index 3d4fe939734d..a03166da973a 100644 --- a/lld/wasm/InputFiles.cpp +++ b/lld/wasm/InputFiles.cpp @@ -691,12 +691,16 @@ StringRef strip(StringRef s) { } void StubFile::parse() { - bool first = false; + bool first = true; + + SmallVector<StringRef> lines; + mb.getBuffer().split(lines, '\n'); + for (StringRef line : lines) { + line = line.trim(); - for (StringRef line : args::getLines(mb)) { // File must begin with #STUB if (first) { - assert(line == "#STUB\n"); + assert(line == "#STUB"); first = false; } @@ -713,10 +717,10 @@ void StubFile::parse() { symbolDependencies[sym] = {}; while (rest.size()) { - StringRef first; - std::tie(first, rest) = rest.split(','); - first = strip(first); - symbolDependencies[sym].push_back(first); + StringRef dep; + std::tie(dep, rest) = rest.split(','); + dep = strip(dep); + symbolDependencies[sym].push_back(dep); } } } |