[PATCH] D24822: [SelectionDAG] Enable division-by-constant optimization for wide types

Alexei Starovoitov via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 08:04:34 PDT 2016


ast added inline comments.

================
Comment at: test/CodeGen/BPF/sdiv_error.ll:3
@@ -2,3 +2,3 @@
 ; RUN: FileCheck %s < %t1
-; CHECK: Unsupport signed division
+; CHECK: Cannot select: t27: i64,i64 = smul_lohi
 
----------------
'cannot select' is unreadable to C programmer comparing to 'Unsupported signed div'.
Usability of BPF is the hardest problem we're facing, so we don't want to lose those error messages. Hence similar to the SDIV error, can you please add the same error for SMUL_LOHI in BPFDAGToDAGISel::Select() ?
Also keeping the same hint "Please convert to unsigned div/mod".
Thanks


https://reviews.llvm.org/D24822





More information about the llvm-commits mailing list