[PATCH] D58593: [ValueTracking] More accurate unsigned add/sub overflow detection

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 27 08:05:31 PST 2019


spatel added a comment.

Nice!

I want to be sure that we're not missing any diffs in the AddOverFlow.ll test file, so I regenerated that:
rL354993 <https://reviews.llvm.org/rL354993>

I'd also prefer that we have the baseline tests for the intrinsics committed to trunk as a preliminary step, so we just have the diffs here. Can you rebase and re-upload the patch here after those changes?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58593/new/

https://reviews.llvm.org/D58593





More information about the llvm-commits mailing list