[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