RE: r274110 - [AVX512] Zero extend cmp intrinsic return value.

2016-07-04 Thread Breger, Igor via cfe-commits
Thanks for the explanation. Regards, Igor From: Craig Topper [mailto:craig.top...@gmail.com] Sent: Monday, July 04, 2016 21:53 To: Breger, Igor Cc: Eric Christopher via cfe-commits; Demikhovsky, Elena Subject: Re: r274110 - [AVX512] Zero extend cmp intrinsic return value. We have to keep the aut

RE: r274110 - [AVX512] Zero extend cmp intrinsic return value.

2016-07-04 Thread Breger, Igor via cfe-commits
Thanks for working on this! Regards, Igor From: Craig Topper [mailto:craig.top...@gmail.com] Sent: Monday, July 04, 2016 10:20 To: Breger, Igor Cc: Eric Christopher via cfe-commits; Demikhovsky, Elena; Ouriel, Boaz Subject: Re: r274110 - [AVX512] Zero extend cmp intrinsic return value. I've modi

RE: r274110 - [AVX512] Zero extend cmp intrinsic return value.

2016-07-04 Thread Breger, Igor via cfe-commits
Hello Craig, In general for correctness we should also change AutoUpgrade code for the cmp intrinsics. But if we handle cmp intrinsic in Clang ( CGBuildin.cpp) , can we completely remove the relevant code from back-end and Auto-Upgrade? Regards, Igor From: Craig Topper [mailto:craig.top...@gma

RE: r274110 - [AVX512] Zero extend cmp intrinsic return value.

2016-07-03 Thread Breger, Igor via cfe-commits
Hello Craig, Thanks a lot for pointing it out to me. I familiar with this problem, we are planning to improve CodeGen to handle with this case in near future. Regards, Igor From: Craig Topper [mailto:craig.top...@gmail.com] Sent: Saturday, July 02, 2016 08:46 To: Breger, Igor; Eric Christophe