[PATCH] D71920: [AST] Refactor propagation of dependency bits. NFC

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 11 11:01:05 PST 2020


sammccall commandeered this revision.
sammccall added a reviewer: ilya-biryukov.
sammccall added a comment.

@ilya-biryukov isn't working on clang fulltime anymore :-(

I'm going to try to pick up the work on RecoveryExpr, starting with this patch.

@rsmith I think this is waiting on your review (sorry if I've missed anything else that was outstanding).

If you'd like to have `concept::Requirement` use a similar bitfield, I'd just like to confirm my understanding of the current code before refactoring it:

- there's just one `Dependent` bit (along with `UnexpandedPack`) - instantiation-dependence isn't relevant?
- RequiresExpr is only instantiation-dependent if value-dependent (unlike other exprs)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71920





More information about the cfe-commits mailing list