[all-commits] [llvm/llvm-project] ea8539: [COFF] Force Symbols containing '.' to be quoted

Keno Fischer via All-commits all-commits at lists.llvm.org
Mon Aug 30 14:31:45 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ea8539111d444ffb93ab09c5865d9a1a0927cb2c
      https://github.com/llvm/llvm-project/commit/ea8539111d444ffb93ab09c5865d9a1a0927cb2c
  Author: Keno Fischer <keno at juliacomputing.com>
  Date:   2021-08-30 (Mon, 30 Aug 2021)

  Changed paths:
    M llvm/lib/IR/Mangler.cpp
    M llvm/test/CodeGen/X86/dllexport.ll

  Log Message:
  -----------
  [COFF] Force Symbols containing '.' to be quoted

In D87099, the mangler learned to quote export directives that contain
special characters. Only alhpanumerical characters as well as
'_', '$', '.' and '@' were exmpt from this quoting. However, at least
binutils considers an unquoted '.' to be syntax and object files
containing such symbols will cause errors during linking. Fix that
by removing '.' from the list of allowed exemptions.

Differential Revision: https://reviews.llvm.org/D100359




More information about the All-commits mailing list