[llvm] [llvm-dlltool] Remove the i386 underscore prefix from COFFImportFile::ImportName. NFC. (PR #98226)
Jacek Caban via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 10 09:30:25 PDT 2024
Martin =?utf-8?q?Storsjö?= <martin at martin.st>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/98226 at github.com>
================
@@ -691,10 +692,15 @@ Error writeImportLibrary(StringRef ImportName, StringRef Path,
}
if (!E.ImportName.empty() && Name != E.ImportName) {
+ StringRef Prefix = "";
+ if (Machine == IMAGE_FILE_MACHINE_I386 && AddUnderscores)
+ Prefix = "_";
----------------
cjacek wrote:
I think this is not exactly the same as the code removed from `COFFModuleDefinition`. The original code also checks if the symbol is not decorated before adding the prefix.
https://github.com/llvm/llvm-project/pull/98226
More information about the llvm-commits
mailing list