<div dir="ltr"><p class="MsoNormal">I'll fix this.</p><p class="MsoNormal"><br></p><p class="MsoNormal">Vladimir</p><p class="MsoNormal"><br></p><p class="MsoNormal">-----------------------------------------------------------------------------------</p><p class="MsoNormal">From: <b>Akira Hatanaka</b> <<a href="mailto:ahatanak@gmail.com">ahatanak@gmail.com</a>><br>
Date: Fri, Sep 2, 2016 at 3:00 AM<br>
Subject: Re: [PATCH] D21678: Fix For pr28288 - Error message in shift of vector
values<br>
To: <a href="mailto:vladimir.1@gmail.com">vladimir.1@gmail.com</a>, <a href="mailto:ulrich.weigand@de.ibm.com">ulrich.weigand@de.ibm.com</a>, <a href="mailto:amjad.aboud@intel.com">amjad.aboud@intel.com</a>, <a href="mailto:guy.benyei@intel.com">guy.benyei@intel.com</a>, <a href="mailto:aaron.ballman@gmail.com">aaron.ballman@gmail.com</a><br>
Cc: <a href="mailto:ahatanak@gmail.com">ahatanak@gmail.com</a>, <a href="mailto:andreybokhanko@gmail.com">andreybokhanko@gmail.com</a>, <a href="mailto:anastasia.stulova@arm.com">anastasia.stulova@arm.com</a>, <a href="mailto:dmitry.polukhin@gmail.com">dmitry.polukhin@gmail.com</a>, <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<br>
<br>
ahatanak added a subscriber: ahatanak.<br>
ahatanak added a comment.<br>
<br>
This patch causes clang to error out on the following code, which used to
compile fine:<br>
<br>
$ cat f2.c<br>
<br>
  typedef __attribute__((__ext_vector_type__(8))) unsigned short
vector_ushort8;<br>
<br>
  vector_ushort8 foo1(void) {<br>
    return 1 << (vector_ushort8){7,6,5,4,3,2,1,0};<br>
  }<br>
<br>
$ clang f2.c -c<br>
<br>
clang used to transform the scaler operand to a vector operand (similar to the
way gcc's vector is handled) when compiling for normal c/c++ (but printed an
error message when compiling for opencl), but this patch dropped the check for
LangOpts added in r230464 and changed that behavior. I don't think this was
intentional?<span></span></p>

<span style="font-size:12pt;font-family:"times new roman",serif"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D21678" target="_blank">https://reviews.llvm.org/D21678</a></span><br></div>