[PATCH] D84405: CodeGen: Improve generated IR for __builtin_mul_overflow(uint, uint, int)
Vedant Kumar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 28 12:10:06 PDT 2020
vsk accepted this revision.
vsk added a comment.
This revision is now accepted and ready to land.
Thanks, looks good to me with a small test addition.
================
Comment at: clang/test/CodeGen/builtins-overflow.c:123
+ // CHECK: br i1 [[C2]]
+ int r;
+ if (__builtin_mul_overflow(x, y, &r))
----------------
vsk wrote:
> Could you add a test for the volatile result case?
I think this is still missing?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84405/new/
https://reviews.llvm.org/D84405
More information about the cfe-commits
mailing list