[PATCH] D70079: [SelectionDAG] Combine U{ADD,SUB}O diamonds into {ADD,SUB}CARRY
David Zarzycki via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 14 00:33:01 PST 2019
davezarzycki added a comment.
Thank you @spatel for adding more reviewers. Questions I have as a non-expert of LLVM design:
1. Why is UADDO/USUBO an intrinsic but ADDCARRY/SUBCARRY is not? This is unfortunate given that clang and perhaps other languages have addcarry/subcarry intrinsics.
2. Is the diamond being updated correctly? Or does it work by accident?
3. Is there a good example of how to break a diamond when updating the DAG?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70079/new/
https://reviews.llvm.org/D70079
More information about the llvm-commits
mailing list