[PATCH] D43446: [X86] Correct SHRUNKBLEND creation to work correctly when there are multiple uses of the condition.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 20 10:42:24 PST 2018


RKSimon added a comment.

Cheers.



================
Comment at: lib/Target/X86/X86ISelLowering.cpp:31909
         DCI.CommitTargetLoweringOpt(TLO);
-        return SDValue();
+        return SDValue(N, 0);
       }
----------------
craig.topper wrote:
> RKSimon wrote:
> > I take it there's no reason to re-evaluate this?
> We don't have any combines on SHRUNKBLEND today. Do you think I should add it to teh worklist anyway?
Probably not worth it - if a use case arises its a trivial change.


Repository:
  rL LLVM

https://reviews.llvm.org/D43446





More information about the llvm-commits mailing list