[flang-commits] [PATCH] D108622: [flang] Implement Posix version of DATE_AND_TIME runtime

Tue Aug 24 07:48:17 PDT 2021

Leporacanthicus added a comment.

In D108622#2962282 <https://reviews.llvm.org/D108622#2962282>, @jeanPerier wrote:

> In D108622#2962219 <https://reviews.llvm.org/D108622#2962219>, @Leporacanthicus wrote:
>
>> Looks like it's doing what's needed. Is there a follow-on patch to fix the fact that `values` isn't being passed to the `DATE_AND_TIME` subroutine?
>
> Do you mean a patch in lowering ?
>
> If so, yes, once this is up streamed, lowering will be updated.

Yes, I mean the lowering of calls to `DATE_AND_TIME` - line 2080 in IntrinsicCall.cpp which prints a message saying values is not yet implemented.

If you remember, do add me to that review.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108622