[PATCH] D139784: [Doc] Refactor descriptions of `min-legal-vector-width`

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