[PATCH] D82662: [CodeGen] Use the common semantic for fixed-point codegen, not the result semantic.

Bevin Hansson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 26 09:17:32 PDT 2020


ebevhan created this revision.
ebevhan added a reviewer: leonardchan.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Using the result semantic is wrong in some cases, such as
unsigned fixed-point + signed integer. In this case, the
result semantic is unsigned and the common semantic is
signed.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D82662

Files:
  clang/lib/CodeGen/CGExprScalar.cpp
  clang/test/Frontend/fixed_point_add.c
  clang/test/Frontend/fixed_point_div.c
  clang/test/Frontend/fixed_point_mul.c
  clang/test/Frontend/fixed_point_sub.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D82662.273754.patch
Type: text/x-patch
Size: 11235 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200626/ffa955a2/attachment.bin>


More information about the cfe-commits mailing list