[PATCH] D43306: [X86] Add pass to infer required-vector-width attribute based on size of function arguments and use of intrinsics

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 7 20:47:08 PDT 2018


craig.topper updated this revision to Diff 141519.
craig.topper added a comment.

-Rename required-vector-width to legal-vector-width.
-Add a command line option to use prefer-vector-width for legal-vector-width if legal-vector-width is not already provided
-Simplify test to mostly just use the command line option so we don't have to repeat the test bodies with different attributes.

The intention here is to work towards removing prefer-vector-width in favor of legal-vector-width. This will require clang changes to set the legal-vector-width attribute. The command line option added here enables experimenting with the backend support while the clang changes are being developed.


https://reviews.llvm.org/D43306

Files:
  lib/Target/X86/X86Subtarget.cpp
  lib/Target/X86/X86Subtarget.h
  lib/Target/X86/X86TargetMachine.cpp
  test/CodeGen/X86/legal-vector-width.ll
  test/CodeGen/X86/required-vector-width.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43306.141519.patch
Type: text/x-patch
Size: 63741 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180408/d98685ef/attachment-0001.bin>


More information about the llvm-commits mailing list