[flang-commits] [PATCH] D136254: [flang] add fir.declare codegen support

[Jean Perier via Phabricator via flang-commits]

jeanPerier added a comment.

In D136254#3868201 <https://reviews.llvm.org/D136254#3868201>, @kiranchandramohan wrote:

> Are the pre-codegen ops enough for carrying the debug info to LLVM dialect? Or would we require lowering the `fir.declare` op to LLVM at some point when we add debug info? Would an alternative path be for `fir.declare` to subsume the precodegen dialect ops?

I think it will likely need to go a bit further, but since I do not know how debug info will look like in LLVM IR dialect, I cannot be assertive and I do not want to add a cg fir.declare at that stage just in case. It should really not be hard to add when the need arise, or to even allow fir.shape to be translated to some llvm tuple value in codegen and keep fir.declare until then. It may also actually turnout to be possible to already generate debug info in precodegen, this really depends on what should be generated.

I would be really happy if an LLVM debug info expert could jump in on the debug info task and set the exact requirement and plan for FIR/LLVM dialect here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136254