[Mlir-commits] [mlir] [MLIR][LLVM] Add DILocAttr for debug locations (PR #186146)

[Tobias Gysi]

gysit wrote:

> Totally understandable. We'll try to address the following things in the RFC:
> 
> 1. Why this change is beneficial
> 2. Decisions made (e.g. using CallSiteLoc rather than `inlined_at`)
> 3. A transition plan for in-tree things that produce LLVM locations
> 
> Anything else/other concerns you think would be good to address in the RFC?

Sounds great! 

The main goal of the RFC is that downstream and in-tree users get an overview and an understanding of the change including its motivation. I think that should be well covered by the bullet list. 


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