[PATCH] D123284: [ArgPromotion][Attributor] Update min-legal-vector-width when do promotion

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 19 00:48:02 PDT 2022


nikic added a comment.

I'm still having a hard time understanding what the intended semantics of this attribute are -- my best guess from this patch is that "min-legal-vector-width" is actually supposed to be a "max-legal-vector-width", which is why we need to increase it during promotion?

Before we start doing any code changes, I think we need a LangRef patch that specifies the precise semantics and requirements of this attribute.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123284



More information about the llvm-commits mailing list