From: Craig Topper Date: Fri, 9 Aug 2019 05:55:41 +0000 (+0000) Subject: [X86] Remove stale FIXME from combineMaskedStore. NFC X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=7a7e03f906aada0cf4b749b51213fe5784eeff84;p=llvm [X86] Remove stale FIXME from combineMaskedStore. NFC I believe PR34584 was tracking that FIXME, but its since been closed and a test case was added. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@368397 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Target/X86/X86ISelLowering.cpp b/lib/Target/X86/X86ISelLowering.cpp index 9138b65fdea..c89eff8ec2c 100644 --- a/lib/Target/X86/X86ISelLowering.cpp +++ b/lib/Target/X86/X86ISelLowering.cpp @@ -39760,10 +39760,6 @@ static SDValue combineMaskedStore(SDNode *N, SelectionDAG &DAG, return SDValue(N, 0); } - // TODO: AVX512 targets should also be able to simplify something like the - // pattern above, but that pattern will be different. It will either need to - // match setcc more generally or match PCMPGTM later (in tablegen?). - SDValue Value = Mst->getValue(); if (Value.getOpcode() == ISD::TRUNCATE && Value.getNode()->hasOneUse() && TLI.isTruncStoreLegal(Value.getOperand(0).getValueType(),