[PATCH] D57219: [Fixed Point Arithmetic] Fixed Point Comparisons
Bevin Hansson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 25 01:51:29 PST 2019
ebevhan added inline comments.
================
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:141
+ (RHSType->isFixedPointType() &&
+ LHSType->isFixedPointOrIntegerType());
+ }
----------------
Can't it just be `LHSType->isFixedPointType() || RHSType->isFixedPointType()`?
I don't think there are cases where one is a fixed-point type and the other is not an integer or another fixed-point type, so if one is fixed-point then you already know it's a fixed-point operation.
================
Comment at: clang/test/Frontend/fixed_point_comparisons.c:56
+
+void TestComparisons() {
+ short _Accum sa;
----------------
Missing saturating and saturating/non-saturating comparisons. I'd like to see the differences between unsigned padding and not there, if there are any.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57219/new/
https://reviews.llvm.org/D57219
More information about the cfe-commits
mailing list