[PATCH 2/2] R600/SI: Use a ComplexPattern for MUBUF stores

Tom Stellard tom at stellard.net
Thu Jun 19 08:54:25 PDT 2014


On Wed, Jun 18, 2014 at 04:27:20PM -0700, Matt Arsenault wrote:
> On 06/18/2014 04:10 PM, Tom Stellard wrote:
> >+  if (CurDAG->isBaseWithConstantOffset(Addr)) {
> >+    ConstantSDNode *C1 = cast<ConstantSDNode>(N1);
> >+
> >+    if (C1 && isUInt<12>(C1->getZExtValue())) {
> You either meant to use dyn_cast, or the C1 check is unneccessary

The check is unnecessary.  Attached is an updated patch.

-Tom
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-R600-SI-Use-a-ComplexPattern-for-MUBUF-stores.patch
Type: text/x-diff
Size: 7397 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140619/84e6e39f/attachment.patch>


More information about the llvm-commits mailing list