[flang-commits] [flang] [Flang] Handle spurious DATA error for PDT character components (PR #213598)

Tarun Prabhu via flang-commits flang-commits at lists.llvm.org
Tue Oct 6 04:43:56 PDT 2026


tarunprabhu wrote:

The of this PR title suggests that this PR fixes the underlying issue, but the first paragraph of the description clarifies that it only emits a better diagnostic. Please fix the title to better reflect what this fix actually does.

The PR description will become the commit message once this is merged. The title of the PR will be the first line in that message. As it is, the description has too much unnecessary detail, most of which is not appropriate for a commit message (some guidelines are provided in LLVM's [developer documentation](https://llvm.org/docs/DeveloperPolicy.html#commit-messages). It is probably sufficient to leave only the first paragraph and the "Fixes" line and omit the rest. You can add a follow-up comment on this PR to provide that detail if you think it will be useful for reviewers. 

I generally don't review PR's related to semantics - hopefully one of the other tagged reviewers can take a look.

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


More information about the flang-commits mailing list