Committed dca23ed; should be fixed now. -Eli
From: Azhar Mohammed <az...@apple.com> Sent: Monday, August 3, 2020 11:01 AM To: Eli Friedman <reviews+d84703+public+194bbb12cc03c...@reviews.llvm.org>; Eli Friedman via Phabricator <revi...@reviews.llvm.org>; Eli Friedman <efrie...@quicinc.com> Cc: sander.desma...@arm.com; sjoerd.mei...@arm.com; t.p.northo...@gmail.com; cameron.mcina...@nyu.edu; blitzrak...@gmail.com; ju...@samsung.com; Anna Welker via Phabricator via llvm-commits <llvm-comm...@lists.llvm.org>; mlek...@skidmore.edu; kanh...@a-bix.com; Jonathan Coe via cfe-commits <cfe-commits@lists.llvm.org>; shen...@google.com; kristof.be...@arm.com; daniel.k...@arm.com Subject: [EXT] Re: [PATCH] D84703: [clang codegen][AArch64] Use llvm.aarch64.neon.fcvtzs/u where it's necessary Hey Eli, Looks like this is causing the test-suite build to fail. Can you please take a look. Refer to http://green.lab.llvm.org/green/job/test-suite-verify-machineinstrs-aarch64-O3/8035/consoleFull. FAILED: SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o /Users/buildslave/jenkins/workspace/test-suite-verify-machineinstrs-aarch64-O3/test-suite-build/tools/timeit --summary SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o.time /Users/buildslave/jenkins/workspace/test-suite-verify-machineinstrs-aarch64-O3/compiler/bin/clang -DNDEBUG -B /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin -Wno-unused-command-line-argument -mllvm -verify-machineinstrs -O3 -arch arm64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS13.5.sdk -w -Werror=date-time -std=c99 -MD -MT SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o -MF SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o.d -o SingleSource/UnitTests/Vector/AArch64/CMakeFiles/aarch64_neon_intrinsics.dir/aarch64_neon_intrinsics.c.o -c /Users/buildslave/jenkins/workspace/test-suite-verify-machineinstrs-aarch64-O3/test-suite/SingleSource/UnitTests/Vector/AArch64/aarch64_neon_intrinsics.c fatal error: error in backend: Cannot select: intrinsic %llvm.aarch64.neon.fcvtzs Thanks Azhar On Jul 30, 2020, at 3:42 PM, Eli Friedman via Phabricator via llvm-commits <llvm-comm...@lists.llvm.org<mailto:llvm-comm...@lists.llvm.org>> wrote: This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG8dfb5d767e70: [clang codegen][AArch64] Use llvm.aarch64.neon.fcvtzs/u where it's necessary (authored by efriedma). Changed prior to commit: https://reviews.llvm.org/D84703?vs=281052&id=282069#toc Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84703/new/ https://reviews.llvm.org/D84703 Files: clang/lib/CodeGen/CGBuiltin.cpp clang/test/CodeGen/aarch64-neon-fcvt-intrinsics.c clang/test/CodeGen/aarch64-neon-intrinsics.c clang/test/CodeGen/aarch64-neon-misc.c clang/test/CodeGen/aarch64-v8.2a-fp16-intrinsics.c clang/test/CodeGen/aarch64-v8.2a-neon-intrinsics.c <D84703.282069.patch>_______________________________________________ llvm-commits mailing list llvm-comm...@lists.llvm.org<mailto:llvm-comm...@lists.llvm.org> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
