[PATCH] D62876: [LLD] [MinGW] Support the -u/--undefined option.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 4 18:35:39 PDT 2019


ruiu added a comment.

This is LGTM, but I believe you will have to implement -u in a more correct way. IIRC, I implemented -u as -require-defined in the ELF linker first. It seemed to be working fine until we found something weird, investigated it and then found that that was caused by an incorrect implementation of -u. So, if it is not too hard, you probably should implement it in the right way.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D62876





More information about the llvm-commits mailing list