[PATCH] D72068: [IR] Refactor SubclassData

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

I feel like this is creating too much unnecessary abstraction. I think we can use simple techniques, like unions, as I did here in this uncommitted patch:
https://reviews.llvm.org/D51664#C1373737NL442
Even better, we can do that incrementally, without many changes to the get/set value subclass data APIs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72068