[PATCH] D112159: Relax assert in ExprConstant to a return None.

[Jon Chesterfield via Phabricator via cfe-commits]

JonChesterfield added a comment.

Patched up the call sites that are straightforward. There are ~4 calls in SemaOpenMP where an `!isValueDependent()` could be dropped if it is safe to call PerformContextualImplicitConversion on such a value, which it probably is, but it's hard to be certain. There are a few places where Diag() calls would be changed by dropping the call, those are also left unchanged here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112159