[all-commits] [llvm/llvm-project] 43d2ef: [flang][lowering] propagate location info of macro...

jeanPerier via All-commits all-commits at lists.llvm.org
Tue Sep 26 11:33:14 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 43d2ef2856fc3373068c020efa11a933477e11fa
      https://github.com/llvm/llvm-project/commit/43d2ef2856fc3373068c020efa11a933477e11fa
  Author: jeanPerier <jperier at nvidia.com>
  Date:   2023-09-26 (Tue, 26 Sep 2023)

  Changed paths:
    M flang/lib/Lower/Bridge.cpp
    A flang/test/Lower/macro-debug-file-loc.f90

  Log Message:
  -----------
  [flang][lowering] propagate location info of macro expansions (#67446)

Currently flang-new -g is failing when compiling code containing a call
in a macro to a function defined in the same file.

The verification added in https://reviews.llvm.org/D157447 is valid,
flang lowering was failing to propagate location information in code
from macro expansion because GetSourcePositionRange does not work with
them (it fails to come with an end location), but we do not need a range
for the MLIR location, only the start.

Use GetSourcePosition instead that works with code from macro expansion.

Note that the source location is the one of the statement where the
macro appeared, if needed some FusedLocation could be later built to
keep a link to the macro location in the debug info.




More information about the All-commits mailing list