[PATCH] D75529: [analyzer] Limit UCharMax to min of max uchar or max int

Csaba Dabis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 9 10:46:54 PDT 2020


Charusso accepted this revision.
Charusso added a comment.

In D75529#1912849 <https://reviews.llvm.org/D75529#1912849>, @vabridgers wrote:

> @Charusso, I cannot find a way to create a test for this change, since this was found using an architecture that supports 16-bit chars.


I really felt that we could write a test that using a target which behaves like that. I am not that awesome with triples, sorry. If I am right we are good to go without @NoQ's response (he is the maintainer of the Analyzer). Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75529





More information about the cfe-commits mailing list