[PATCH] D155623: [AArch64][NFC] Expand coverage of ReplaceWithVeclib testing using SLEEF vector library
    mgabka via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jul 24 03:23:30 PDT 2023
    
    
  
mgabka accepted this revision.
mgabka added a comment.
This revision is now accepted and ready to land.
Thanks for updating the patch, it LGTM.
I realised one more thing, the "fast" flag attached to the calls in the tests is not needed.
The replace-with-veclib pass does not check for it and and performs the transformation always when the mappings exist.
However in this pass there is a chunk of code ensuring that the fast math flags are preserved, so I think we can keep it in your tests as it is.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155623/new/
https://reviews.llvm.org/D155623
    
    
More information about the llvm-commits
mailing list