[PATCH] D114241: [fir] Add !fir.len_param_index conversion

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 22 04:05:44 PST 2021


rovka added a comment.

In D114241#3145739 <https://reviews.llvm.org/D114241#3145739>, @awarzynski wrote:

> It sounds like this Op is incomplete/incorrect and not needed. I would be in favor of turning this into a `TODO` - otherwise it's just too confusing. But it will only makes sense if we do the same on `fir-dev`. WDYT?

SGTM.

I'll send a separate patch for the verifier/docs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114241



More information about the llvm-commits mailing list