[PATCH] D137558: [clang][Interp] Reject reinterpret_casts

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 10 13:58:25 PST 2022


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



More information about the cfe-commits mailing list