[PATCH] D85128: [Prototype][SVE] Support arm_sve_vector_bits attribute

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 4 10:41:55 PDT 2020


efriedma added a comment.

> I guess with what you're suggesting the bitcast could still be emitted there but the cast operations could be limited in Sema to cases where ultimately ConvertType would return a type that requires bitcasting, or are you saying that could be avoided completely?

The bitcast operation would exist in Sema either way; it's necessary for the types to stay consistent.  My suggestion is just that bitcasting between a VLAT and the corresponding VLST it would be a no-op in CodeGen.

My suggestion is similar to the way the "bool" type works: in memory, it's an i8, but when you load it, it's truncated it to i1.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85128



More information about the llvm-commits mailing list