[PATCH] D41910: [Concepts] Constrained partial specializations and function overloads.

[Saar Raz via Phabricator via cfe-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rG12038be20ee6: [Concepts] Fix crash in D41910 (authored by saar.raz).

Changed prior to commit:
  https://reviews.llvm.org/D41910?vs=232457&id=234587#toc

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D41910

Files:
  clang/lib/Sema/SemaConcept.cpp


Index: clang/lib/Sema/SemaConcept.cpp
===================================================================
--- clang/lib/Sema/SemaConcept.cpp
+++ clang/lib/Sema/SemaConcept.cpp
@@ -558,7 +558,7 @@
     Atomic.ParameterMapping.emplace();
     Atomic.ParameterMapping->reserve(OccurringIndices.size());
     for (unsigned I = 0, C = TemplateParams->size(); I != C; ++I)
-      if (OccurringIndices[I])
+      if (I < OccurringIndices.size() && OccurringIndices[I])
         Atomic.ParameterMapping->push_back(
             S.getIdentityTemplateArgumentLoc(TemplateParams->begin()[I],
                 // Here we assume we do not support things like


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41910.234587.patch
Type: text/x-patch
Size: 646 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191218/3933348d/attachment.bin>