spatel accepted this revision. spatel added a comment. This revision is now accepted and ready to land.
LGTM - I think the odds of code that is using intrinsic vector abs inter-mixing with auto-vectorized code using the abs idiom are small. I noticed one other LLVM codegen test that might want to be updated to use the IR intrinsics to be more relevant -- llvm-project/llvm/test/CodeGen/X86/combine-abs.ll. That can be done with this patch or later. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87101/new/ https://reviews.llvm.org/D87101 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
