<div dir="ltr">This small patch is extracted from <a href="http://reviews.llvm.org/D8943">http://reviews.llvm.org/D8943</a> and the test case is also in that patch. As requested by hfinkel I commit this small patch ahead of that larger one. Could you please take a look as that patch? Thanks!<div><br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature">thanks,<br>Cong</div></div>
<br><div class="gmail_quote">On Wed, Oct 28, 2015 at 11:21 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 28, 2015 at 11:15 AM, Cong Hou via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: conghou<br>
Date: Wed Oct 28 13:15:46 2015<br>
New Revision: 251538<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=251538&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=251538&view=rev</a><br>
Log:<br>
[X86] A small fix in X86/X86TargetTransformInfo.cpp: check a value type is simple before calling getSimpleVT().<br></blockquote><div><br></div><div>Test case?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Modified:<br>
    llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp<br>
<br>
Modified: llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp?rev=251538&r1=251537&r2=251538&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp?rev=251538&r1=251537&r2=251538&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp (original)<br>
+++ llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp Wed Oct 28 13:15:46 2015<br>
@@ -895,8 +895,9 @@ int X86TTIImpl::getMaskedMemoryOpCost(un<br>
<br>
   // Legalize the type.<br>
   std::pair<int, MVT> LT = TLI->getTypeLegalizationCost(DL, SrcVTy);<br>
+  auto VT = TLI->getValueType(DL, SrcVTy);<br>
   int Cost = 0;<br>
-  if (LT.second != TLI->getValueType(DL, SrcVTy).getSimpleVT() &&<br>
+  if (VT.isSimple() && LT.second != VT.getSimpleVT() &&<br>
       LT.second.getVectorNumElements() == NumElem)<br>
     // Promotion requires expand/truncate for data and a shuffle for mask.<br>
     Cost += getShuffleCost(TTI::SK_Alternate, SrcVTy, 0, nullptr) +<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>
</blockquote></div><br></div></div>