shafik added a comment.

In D137558#3915350 <https://reviews.llvm.org/D137558#3915350>, @aaron.ballman 
wrote:

> This is lacking test coverage. (Other interesting related test coverage are 
> casts that perform a reinterpret_cast under the hood but that might be 
> somewhat separate.)

Well I think at least things like C-style casts that basically end up being 
`reinterpret_cast` should be tested as well e.g.:

  constexpr int x = 1;
  constexpr float *fp = (float*)x;


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137558

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D137558: [clang][I... Aaron Ballman via Phabricator via cfe-commits
    • [PATCH] D137558: [cla... Shafik Yaghmour via Phabricator via cfe-commits

Reply via email to