[PATCH] D67051: [LLD] [COFF] Demangle itanium symbols in mingw mode

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 1 14:28:48 PDT 2019


mstorsjo created this revision.
mstorsjo added reviewers: ruiu, rnk, thakis.
Herald added a subscriber: erik.pilkington.
Herald added a project: LLVM.

Maybe the msvc demangling should be within if (!mingw)? Performance wise it's probably irrelevant, but the itanium demangling, in combination with dllimport and i386 leading underscores, ended up as a few more lines, so an if (mingw) around it makes it a bit clearer.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D67051

Files:
  COFF/Symbols.cpp
  test/COFF/undefined-symbol-itanium-i386.s
  test/COFF/undefined-symbol-itanium.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67051.218273.patch
Type: text/x-patch
Size: 3619 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190901/1d4e5722/attachment.bin>


More information about the llvm-commits mailing list