[all-commits] [llvm/llvm-project] b0979b: [ORC] Add optional context string to duplicate sym...

Lang Hames via All-commits all-commits at lists.llvm.org
Fri May 9 00:31:07 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b0979b8c65d76cc1897e97b9ad091d8d99abdd18
      https://github.com/llvm/llvm-project/commit/b0979b8c65d76cc1897e97b9ad091d8d99abdd18
  Author: Lang Hames <lhames at apple.com>
  Date:   2025-05-09 (Fri, 09 May 2025)

  Changed paths:
    M llvm/include/llvm/ExecutionEngine/Orc/Shared/OrcError.h
    M llvm/lib/ExecutionEngine/Orc/Core.cpp
    M llvm/lib/ExecutionEngine/Orc/Shared/OrcError.cpp
    A llvm/test/ExecutionEngine/JITLink/Generic/Inputs/main-ret-42.ll
    A llvm/test/ExecutionEngine/JITLink/Generic/duplicate-definition-error.test

  Log Message:
  -----------
  [ORC] Add optional context string to duplicate symbol definition errors.

The context string can be added to indicate the source of the duplicate
definition. E.g. if the context is set to "foo2.o", then:

"Duplicate definition of symbol 'foo'"

becomes

"In foo2.o, duplicate definition of symbol 'foo'".

The JITDylib::defineImpl method is updated to use the name of the
MaterializationUnit being added as the context string for duplicate definition
errors. The JITDylib::defineMaterializing method is updated to use
"defineMaterializing operation" as the conext string.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list