[PATCH] D60055: Check i < FD->getNumParams() before querying

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 9 05:06:41 PDT 2019


lebedev.ri resigned from this revision.
lebedev.ri added a comment.

I won't be able to properly review this, sorry.
While i'm sure this fix silences the assert, i don't know what that means for the original caller.
Does it fail? Does it retry in some other scope? Should it be more picky about the original scope in the first place?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60055





More information about the cfe-commits mailing list