[Mlir-commits] [mlir] [MLIR] Translate DIStringType. (PR #94480)

[Tobias Gysi]

https://github.com/gysit commented:

Nice!

It looks like LLVM actually has a DIVariable class in its class hierarchy. In that case I would vote for replicating LLVM's hierarchy. We already have similar base classes such as DIType or DIScope. I think DIVariable can be implemented similarly. 

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