[PATCH] D90928: [OpenCL] Add assertions to extension lookup

erik2020 updated this revision to Diff 306094. erik2020 added a comment. Changed asserting code to return `false` instead. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90928/new/ https://reviews.llvm.org/D90928 Files: clang/include/clang/Basic/

[PATCH] D90928: [OpenCL] Add assertions to extension lookup

Anastasia added a comment. In D90928#2382111 , @erik2020 wrote: > In D90928#2379322 , @Anastasia wrote: > >> Ok, it would still segfault but perhaps it is acceptable as this is an >> internal frontend only option f

[PATCH] D90928: [OpenCL] Add assertions to extension lookup

erik2020 added a comment. In D90928#2379322 , @Anastasia wrote: > Ok, it would still segfault but perhaps it is acceptable as this is an > internal frontend only option for now. Would it be better if these functions returned `false` for unknown extension

[PATCH] D90928: [OpenCL] Add assertions to extension lookup

Anastasia added a comment. Ok, it would still segfault but perhaps it is acceptable as this is an internal frontend only option for now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D90928/new/ https://reviews.llvm.org/D90928

[PATCH] D90928: [OpenCL] Add assertions to extension lookup

erik2020 created this revision. erik2020 added a reviewer: ldrumm. erik2020 added a project: clang. Herald added subscribers: cfe-commits, dexonsmith, Anastasia, yaxunl. erik2020 requested review of this revision. Calling any of the `OpenCLOptions::is*()` functions (except `isKnown()`) with an un