jvesely added a comment. `cl_khr_byte_addressable_stores` changes language semantics. without it, pointer dereferences of types smaller than 32 bits are illegal. Even if all clang targets support this the macro should still be defined for backward compatibility.
it would be useful if the commit message or the description of this revision included a justification for each removed extension why it doesn't impact language semantics with spec references. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89372/new/ https://reviews.llvm.org/D89372 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
