[PATCH] D139784: [Doc] Refactor descriptions of `min-legal-vector-width`
Phoebe Wang via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Dec 17 08:08:49 PST 2022
pengfei added a comment.
I want to remove the description from LangRef directly since we decided to only support this attribute in X86. WDYT?
================
Comment at: llvm/docs/LangRef.rst:2241
+ generator will not turn vector types with the same or below size illegal.
+ The value is calculated from the vector arguments and returnings in the
+ function and functions directly called by this function.
----------------
arsenm wrote:
> return types?
>
> Also what is the behavior for aggregates? Aggregates containing vectors? Nested aggregates with vectors?
Good question. I think we have considered such cases. See https://github.com/llvm/llvm-project/blob/main/clang/lib/CodeGen/CGCall.cpp#L4729-L4740
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139784/new/
https://reviews.llvm.org/D139784
More information about the llvm-commits
mailing list