[PATCH] D95458: [PowerPC] Exploit xxsplti32dx (constant materialization) for scalars

Stefan Pintilie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 12 04:24:23 PST 2021


stefanp accepted this revision.
stefanp added a comment.
This revision is now accepted and ready to land.

Thank you for adding this!
Other than one minor nit I think this LGTM.

Feel free to address nits on commits.



================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:8834
+                           &LosesInfo);
+  bool Success = (!LosesInfo && !APFloatToConvert.isDenormal());
+
----------------
nit:
You can just inline this. It is only used in one place.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95458/new/

https://reviews.llvm.org/D95458



More information about the cfe-commits mailing list