[PATCH] D55844: [Fixed Point Arithmetic] Fixed Point Subtraction

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 9 16:06:32 PST 2019


rjmccall added inline comments.


================
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3451
+  case BO_Comma:
+    llvm_unreachable("Found unimplemented fixed point binary operation");
   }
----------------
Please create a separate case for the non-arithmetic operators (pointer-to-member, assignment, comma) that will obviously never end up in this function.

Otherwise, LGTM.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55844





More information about the cfe-commits mailing list