This code also has a lot of 'else' after a block ending in 'return'. This should be avoided per coding standards.<br><br><div class="gmail_quote">On Mon, Dec 3, 2012 at 9:05 AM, Jyotsna Verma <span dir="ltr"><<a href="mailto:jverma@codeaurora.org" target="_blank">jverma@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Let me give it a thought. There is a lot of common functionality across<br>
these predicates. It would be nice to have that factored out somehow. At<br>
this point, I'm going to open a bug and revisit it later.<br>
<br>
Thanks,<br>
Jyotsna<br>
--<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by<br>
The Linux Foundation<br>
<br>
<br>
> -----Original Message-----<br>
> From: Sean Silva [mailto:<a href="mailto:silvas@purdue.edu">silvas@purdue.edu</a>]<br>
> Sent: Monday, December 03, 2012 1:26 AM<br>
> To: Jyotsna Verma<br>
> Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> Subject: Re: [llvm-commits] [llvm] r169117 -<br>
> /llvm/trunk/lib/Target/Hexagon/HexagonOperands.td<br>
><br>
> > Sean, what exactly are you suggesting?<br>
><br>
> I'm not intimately familiar with this part of LLVM, so I don't have any<br>
concrete<br>
> suggestions, but is there not some way to have all these patterns inherit<br>
> from a common base that encapsulates the subtarget check?<br>
><br>
> -- Sean Silva<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>~Craig<br>