aaron.ballman added a comment.
Herald added a project: clang-tools-extra.

> However this has opened a can of worms. Currenty float a = 
> std::numeric_limits<double>::infinity(); is marked as narrowing.

I think this is technically not narrowing, per 
http://eel.is/c++draft/dcl.init.list#7.2, because the source is a constant 
expression (at least, after C++11) and the value can be represented after 
conversion. However, I wonder if the same is true for the various NaNs as it is 
for infinity?

> Whats more suspicious is double a = std::numeric_limits<float>::infinity(); 
> is also marked as narrowing.

This is definitely not narrowing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98416

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

Reply via email to