amyk added a comment. A question I have is, I noticed there are some test case changes involving `xxspltidp` and `xxsplti32dx`, but are additional tests needed for `xxspltiw`?
================ Comment at: llvm/lib/Target/PowerPC/PPCInstrPrefix.td:1887 // XXSPLI32DX needs extra flags to make sure the compiler does not attempt // to spill part of the instruction when the values are similar. ---------------- nit: Update this comment to include `xxspltiw` and `xxspltidp`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108823/new/ https://reviews.llvm.org/D108823 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
