[PATCH] D58346: [Sema] Change addr space diagnostics in casts to follow C++ style

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 4 22:25:32 PST 2019


rjmccall added a comment.

In the abstract, it would be nice to warn about casts that always have UB because the address spaces don't overlap; however, I'm worried about how people might be using address spaces in strange ways today, knowing that they *do* overlap, only in ways that the compiler isn't smart enough for.  I think we should just be permissive in non-OpenCL mode.


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

https://reviews.llvm.org/D58346





More information about the cfe-commits mailing list