[llvm] [MCP] Move dependencies if they block copy propagation (PR #105562)

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 23 04:18:44 PDT 2024


qcolombet wrote:

> I think I could inline the lambda in the BFS shall I do it?

I'm not sure what this would look like, but generally speaking I like subroutines (I even prefer static functions to lambdas) because it makes the contract between the functionality and the surrounding code more obvious and forces you to document the API (what the input arguments are and what is returned mean!)

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


More information about the llvm-commits mailing list