[all-commits] [llvm/llvm-project] c73516: [ORC] Add tests for error handling paths in suspen...

Lang Hames via All-commits all-commits at lists.llvm.org
Thu May 9 06:25:12 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c73516af10f800d0d5641651eefce128b866a155
      https://github.com/llvm/llvm-project/commit/c73516af10f800d0d5641651eefce128b866a155
  Author: Lang Hames <lhames at gmail.com>
  Date:   2024-05-09 (Thu, 09 May 2024)

  Changed paths:
    M llvm/unittests/ExecutionEngine/Orc/CoreAPIsTest.cpp

  Log Message:
  -----------
  [ORC] Add tests for error handling paths in suspended generators.

Test that (1) errors returned from a manually suspended generator are
propagated as expected, and (2) automatic suspension does not interfere with
our ability to resume (and return errors from) a generator.


  Commit: cbf1535cc813b2f226498d974c43675832abc233
      https://github.com/llvm/llvm-project/commit/cbf1535cc813b2f226498d974c43675832abc233
  Author: Lang Hames <lhames at gmail.com>
  Date:   2024-05-09 (Thu, 09 May 2024)

  Changed paths:
    M llvm/lib/ExecutionEngine/Orc/EPCGenericDylibManager.cpp

  Log Message:
  -----------
  [ORC] Fix fall-through in error case in EPCGenericDylibManager::lookupAsync.

In the event of a serialization error (e.g. due to a network dropout) we should
only run the Complete handler once, passing the serialization error value.

No test-case: This would require a deliberately injected failure in a
remote-JIT test and we don't have the infrastructure for that at the moment.

rdar://126772381


Compare: https://github.com/llvm/llvm-project/compare/0c0fc9a7c6e2...cbf1535cc813

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