[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