[PATCH] D95458: [PowerPC] Exploit xxsplti32dx (constant materialization) for scalars
    Stefan Pintilie via Phabricator via llvm-commits 
    llvm-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 llvm-commits
mailing list