[Lldb-commits] [PATCH] D122041: [llvm][utils] Fix llvm::Optional summary provider

[Dave Lee via Phabricator via lldb-commits]

kastiglione added a comment.

> There's some CMake trickery

Can you shed some more light on this? Am I understanding right: these formatters are llvm, but the test would be in lldb? It seems weird that something in llvm/ would only have tests in lldb/. Would it be bad to move this file into lldb, and then we can test there? With llvm being a monorepo, a source checkout should have access at any path. Maybe the issue would be llvm installations and packages that don't include lldb too?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122041/new/

https://reviews.llvm.org/D122041