[PATCH] D35635: Optimize {s,u}{add,sub}.with.overflow on ARM

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 3 17:35:02 PDT 2017


efriedma added a comment.

Is it really necessary to have two different of almost identical code to generate an ARMISD::BRCOND?  (I would rather have an explicit check for an AND than two versions of the code, if that's the issue.)


https://reviews.llvm.org/D35635





More information about the llvm-commits mailing list