[llvm] [ORC][COFF] Remove the `ExecutionSession&` argument to `COFFPlatform` factory & constructor (PR #112419)

Tyler Kenney via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 16 10:56:30 PDT 2024


tjk213 wrote:

I confirmed that this test passes on my branch on a `c7g.16xlarge` graviton3 instance:
```
build-release [tjk213/coff-platform-refactor] ❯ make check-lldb-api-functionalities-thread-concurrent_events -j
llvm-lit: /home/tyler/workspace/llvm-project/lldb/test/API/lit.cfg.py:192: warning: Could not set a default per-test timeout. Requires the Python psutil module but it could not be found. Try installing it via pip or via your operating system's package manager.
llvm-lit: /home/tyler/workspace/llvm-project/lldb/test/API/lit.cfg.py:125: note: Deleting module cache at /home/tyler/workspace/llvm-project/llvm/build-release/lldb-test-build.noindex/module-cache-lldb/lldb-api.

Testing Time: 3.33s

Total Discovered Tests: 36
  Passed: 36 (100.00%)

1 warning(s) in tests
[100%] Built target check-lldb-api-functionalities-thread-concurrent_events
```

...so I don't think this issue is caused by my change. Please let me know if there is anything else I need to do.

https://github.com/llvm/llvm-project/pull/112419


More information about the llvm-commits mailing list