[Mlir-commits] [lldb] [llvm] [mlir] [DebugInfo] Add symbolic branches to DIExpression (PR #210850)

[Felipe de Azevedo Piovezan]

felipepiovezan wrote:

> It's also not quite clear to me why RFC is necessary for this particular patch, or is it more general RFC to talk about how overall procedures will be implemented?

Well, there are clear design decisions that were made unilaterally here, and there are valid open questions about future direction. We can keep iterating on those over github, but that's not the best forum.

> What would be the benefit of splitting the patch?

Easier to review, easier to revert, easier to see how things are decoupled. This is the same question of "why are smaller patches better?", no?

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