[PATCH] D53137: Scalable vector core instruction support + size queries

David Greene via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 07:19:28 PDT 2019


greened added a comment.

In D53137#1652189 <https://reviews.llvm.org/D53137#1652189>, @rovka wrote:

>




> +1 from me, I definitely like Sander's suggestion. I thought we were heading towards ScalableSize-only functions anyway and that this was just a temporary path until we cleaned up the codebase from unsigned. Using a cast operator from the start seems like an even better way of getting there, since that way at least we hopefully won't get that many new uses of unsigned in the interim.

+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D53137





More information about the llvm-commits mailing list