[PATCH] D71989: [OpenMP][IRBuilder][WIP] Prototype `omp task` support

Shraiysh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 28 23:09:33 PDT 2022


shraiysh added a comment.

In D71989#3413239 <https://reviews.llvm.org/D71989#3413239>, @jdoerfert wrote:

> In D71989#3413237 <https://reviews.llvm.org/D71989#3413237>, @shraiysh wrote:
>
>> Hi all, is this patch being worked on? I wanted to use this for adding support for task construct in flang.
>
> I don't think it is. Read the discussion to see why this got stalled. Best way forward is to take this but *not* do the API changes I included.

Okay, I am assuming a lot of the OpenMPIRBuilder.cpp code can be reused from here, but instead of creating the new proposed function, I should create the old functions with those arguments. I hope this is the correct approach, please correct me if I am wrong.

I wanted to know how to proceed about this? Do I borrow stuff from this patch and submit a new differential with you as a co-author? Do I push to this same differential? (Can we do that?) Please advise me on this. Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71989



More information about the llvm-commits mailing list