ABataev added a comment.

In D98135#2617446 <https://reviews.llvm.org/D98135#2617446>, @lxfind wrote:

> In D98135#2617432 <https://reviews.llvm.org/D98135#2617432>, @ABataev wrote:
>
>> There is a problem. We actually do not emit `S` here directly, instead, we 
>> use `CodeGen` lambdas, which may not be equal to `S`, in some cases `S` is 
>> `nullptr` here. It may result in not quite accurate results.
>
> Thanks for the note! 
> I don't really know anything about OMP though, not sure how to handle it. 
> Would you mind taking a look at this issue? Feel free to send a different 
> patch!

Could you create a PR for the problem so we could track it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98135

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to