aaron.ballman added inline comments.

================
Comment at: clang/test/Sema/warn-literal-range.c:25
+
+double d5 = 0x0.42p+4200; // expected-warning {{magnitude of floating-point 
constant too large for type 'double'; maximum is 1.7976931348623157E+308}}
----------------
I think we should probably have test coverage for `long double` as well, but I 
also wonder whether it makes sense to add coverage for the small floating-point 
types (like `_Float16`) as well, or whether we already have coverage for those 
elsewhere.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119528

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to