[PATCH] D150605: LLT: Add some stub constructors for FP types

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 00:36:54 PDT 2023


arsenm added a comment.

In D150605#4344853 <https://reviews.llvm.org/D150605#4344853>, @Pierre-vh wrote:

> However if we're going this direction I would prefer to see all relevant uses of `LLT::scalar` looked at and replaced with `::float` where needed (not just within the AMD backend) to avoid further fragmentation.

This isn't a complete set for AMDGPU either, and I don't see how it's fragmenting since this is just documenting at this point. Documenting FP uses like this will allow incrementally moving to a state where it's easier to introduce the new types


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

https://reviews.llvm.org/D150605



More information about the llvm-commits mailing list