vbvictor wrote:

> Decide whether to provide an autofix. I have chosen not to, because it will 
> depend on the context one is using a type in.

I don't think we need any fixes for this check - it's a tricky field, and we 
should let the user decide how to resolve these warnings.

> Decide whether float and double should be linted in favour of [stdfloat 
> types](https://en.cppreference.com/w/cpp/types/floating-point.html) or 
> whether this should be a separate check.

I hope `float` types will be an option for this check in the future. Even 
better if we could have 2 options - one to enable integers, the other to enable 
floats. To achieve this, we could the check name to e.g. 
`portability-avoid-platform-specific-fundamental-types` in advance.



https://github.com/llvm/llvm-project/pull/146970
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to