pengfei added a comment.

In D139701#3993446 <https://reviews.llvm.org/D139701#3993446>, @arsenm wrote:

> TargetCodeGenInfo::setTargetAttributes already exists as a place for targets 
> to emit their custom attributes, x86 could emit it there

It's an attractive suggestion. But it's not easy to move it into target code 
because the value is not a static one but calculated during codegen. As I 
explained in LangRef, the value is `max(maxVectorWidthInArgsRet(CurFn), 
maxVectorWidthInArgsRet(Callee0), maxVectorWidthInArgsRet(Callee1), ...)`. We 
may need to copy/paste a lot of codegen code there to achieve the same goal.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139701

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to