[PATCH] D111337: [fir] Add array value copy pass

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 8 01:05:48 PDT 2021


clementval added a comment.

In D111337#3050066 <https://reviews.llvm.org/D111337#3050066>, @mehdi_amini wrote:

> That is.... large. I can't really review this.
>
> I'm curious about what a test-coverage would show on this? What is the proportion of uncovered code here?

Obviously the test (together with this patch) does not cover 100% of the code. I heard that it was discussed that some of the fir-dev upstream patches would be bigger. I can of course work more on this patch and strip done what is not used at the moment. It will come back later with the lowring and the codegen. Let me know what works better for you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111337



More information about the llvm-commits mailing list