Re: [PATCH] D12212: [Headers][X86] Add -O0 assembly tests for intrinsics.

2015-08-25 Thread Ahmed Bougacha via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL245987: [Headers][X86] Add -O0 assembly tests for avx2 intrinsics. (authored by ab). Changed prior to commit: http://reviews.llvm.org/D12212?vs=32732id=33147#toc Repository: rL LLVM

Re: [PATCH] D12212: [Headers][X86] Add -O0 assembly tests for intrinsics.

2015-08-20 Thread Simon Pilgrim via cfe-commits
RKSimon added a comment. Definitely like this idea - hopefully it'll make it safer to remove unnecessary builtins. Comment at: avx2-builtins-codegen.c:182 @@ -151,1 +181,3 @@ + // FIXME-CHECK-ASM: vpxor %ymm{{.*}} + // FIXME-CHECK-ASM: vandps {{.*}}, %ymm{{.*}} return