[PATCH] D98847: [flang][driver] Fix `-fdebug-dump-provenance`

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 18 02:48:00 PDT 2021


awarzynski created this revision.
Herald added a reviewer: sscalpone.
awarzynski requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The `-fdebug-dump-provenance` flag is meant to be used with
`needProvenanceRangeToCharBlockMappings` set to `true`. This way, extra
mapping is generated that allows e.g. IDEs to retrieve symbol's scope
(offset into cooked character stream) based on symbol's source code
location. This patch makes sure that this option is set when
using `-fdebug-dump-provenance`.

With this patch, the implementation of  `-fdebug-dump-provenance` in
`flang-new -fc1` becomes consistent with `f18`. The corresponding LIT
test is updated so that it can be shared with `f18`. I refined it a bit
so that:

- it becomes a frontend-only test
- it's stricter about the expected output


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98847

Files:
  flang/include/flang/Frontend/FrontendOptions.h
  flang/lib/Frontend/CompilerInvocation.cpp
  flang/test/Driver/debug-provenance.f90

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98847.331498.patch
Type: text/x-patch
Size: 3997 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210318/de1d9a74/attachment.bin>


More information about the llvm-commits mailing list