[PATCH] D149611: [LLD][COFF] Allow overwriting directives exports with cmd-line exports

Thu May 4 08:03:32 PDT 2023

alvinhochun added inline comments.

================
Comment at: lld/test/COFF/ordinals-override.test:65
+    Alignment:       1
+    SectionData:     2020202F6578706F72743A62617A3D3F62617A4040594158585A202F44454641554C544C49423A224C4942434D5422202F44454641554C544C49423A224F4C444E414D455322202F4558504F52543A3F666F6F4040594158585A202F4558504F52543A3F6261724040594158585A20
+  - Name:            '.debug$S'
----------------
mstorsjo wrote:
> Can you add a comment what the contained directive string is?
This is marked "done" but I don't see the comment. Anyway, decoding this shows `/export:baz=?baz@@YAXXZ /DEFAULTLIB:"LIBCMT" /DEFAULTLIB:"OLDNAMES" /EXPORT:?foo@@YAXXZ /EXPORT:?bar@@YAXXZ`, which seems to be why the pre-merge checks failed on Debian, so I guess they should be cleaned up a bit.

```
lld-link: error: could not open 'LIBCMT.lib': No such file or directory
lld-link: error: could not open 'OLDNAMES.lib': No such file or directory
```

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149611