[PATCH] D105423: Add support for Opaque as a LowLevelType

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 8 06:53:20 PDT 2021


arsenm added a comment.

In D105423#2864145 <https://reviews.llvm.org/D105423#2864145>, @pmatos wrote:

> OK - I feel like this became an easier diff to deal with by doing opaques as scalar of size 0. However, to be honest I am not totally convinced yet it's useful to introduce the idea of "opaque" here. Just like we create MVT's that are `Fixed(0)` maybe we should just allow LLT's to have `scalar(0)` calls and remove this "opaque" concept here. What do others think?

That would be a bit nicer than taking a bit just for this. However some places are relying on invalid LLTs that are all zeroes currently


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105423



More information about the llvm-commits mailing list