[PATCH] [X86] replace vinsertf128 intrinsics with generic shuffles

Sanjay Patel spatel at rotateright.com
Thu Mar 5 12:30:59 PST 2015


Hi chandlerc, craig.topper, qcolombet,

This is the sibling patch for the LLVM half of this change:
http://reviews.llvm.org/D8086

We want to remove as much custom x86 shuffling nonsense as possible.

For the avxintrin.h header, it's not clear to me how we decide between macro implementations and inline function implementations. I thought macros were generally frowned upon, so I went with inlines.

http://reviews.llvm.org/D8088

Files:
  include/clang/Basic/BuiltinsX86.def
  lib/Headers/avxintrin.h
  lib/Sema/SemaChecking.cpp
  test/CodeGen/avx-shuffle-builtins.c
  test/CodeGen/builtins-x86.c

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8088.21300.patch
Type: text/x-patch
Size: 5309 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150305/f336b5fc/attachment.bin>


More information about the cfe-commits mailing list