[PATCH] D90173: [PowerPC] Exploit splat instruction xxsplti32dx in Power10
Albion Fung via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 3 10:56:25 PST 2020
Conanap added a comment.
addressed a comment
================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:9366
+ SplatNode = DAG.getNode(
+ PPCISD::XXSPLTI32DX, SDLoc(SplatNode), MVT::v2i64, SplatNode,
+ DAG.getTargetConstant(isLE ? 1 : 0, SplatNode, MVT::i32),
----------------
amyk wrote:
> I think I'm still a little confused by this. Do we not need `dl` when we do `getNode()` here?
Hey so the DL is supplied from `SplatNode`'s definition, and then we pass in `SDLoc(SplatNode)` here as the proper SDLoc after new instr def.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90173/new/
https://reviews.llvm.org/D90173
More information about the llvm-commits
mailing list