[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