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

Mon Nov 9 02:03:21 PST 2020

erik2020 added a comment.

In D90928#2379322 <https://reviews.llvm.org/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 extensions? I think it would be consistent with the function names (e.g., `isEnabled()` returns `false` for an unknown extensions, because an unknown extension cannot be enabled).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90928