craig.topper added a comment. In D117791#3258785 <https://reviews.llvm.org/D117791#3258785>, @RKSimon wrote:
> In D117791#3258602 <https://reviews.llvm.org/D117791#3258602>, @craig.topper > wrote: > >> Just to confirm since I can't see the test CHECK lines. >> builtin_elementwise_abs is not undefined for INT_MIN? > > That's why I added a copy of the check (from > llvm-project\clang\test\CodeGen\X86\avx2-builtins.c) in the summary - the > 'is_int_min_poison' flag in the abs intrinsic call is still false I shouldn't try to read patches too early in the morning I guess. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117791/new/ https://reviews.llvm.org/D117791 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
