[PATCH] D120460: [flang] Fix lowering OpenMP/OpenACC declarative constructs in non-module unit

Shraiysh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 28 06:21:19 PST 2022


shraiysh added a comment.

In D120460#3347208 <https://reviews.llvm.org/D120460#3347208>, @peixin wrote:

> In D120460#3347185 <https://reviews.llvm.org/D120460#3347185>, @kiranchandramohan wrote:
>
>> Can you check why the patch application fails?
>
> This is based on D120459 <https://reviews.llvm.org/D120459>. I had thought the whole file will be applied when applying the patch in Phabricator. I need to rebase this after D120459 <https://reviews.llvm.org/D120459>. Maybe I can merge them if it is easy for review?

Added edge to that revision. This should help in patch application. For future ref, we can add a "Depends on DXXXXXX" line to the summary to automatically create this edge as mentioned here <https://llvm.org/docs/Phabricator.html#using-patch-summaries>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120460



More information about the llvm-commits mailing list