[PATCH] D41944: [LLVM][IR][LIT] support of 'no-overflow' flag for sdiv\udiv instructions
Nuno Lopes via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 17 08:53:51 PST 2018
nlopes added a comment.
Before accepting this patch, we really need to see benchmark results. I'm not going to change clang to start emitting non-UB divs if the perf is going to be horrible. We need data.
Otherwise I don't see the need for this poison version of division. Could you elaborate if your plan is to expose this somehow to the application developer?
I'm sorry if this questions have been properly answered in the past. If so, could you please link them here?
https://reviews.llvm.org/D41944
More information about the llvm-commits
mailing list