[PATCH] D59214: [clang][OpeMP] Model OpenMP structured-block in AST (PR40563)

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 14 09:23:39 PDT 2019


lebedev.ri added a comment.

In D59214#1429384 <https://reviews.llvm.org/D59214#1429384>, @riccibruno wrote:

> IIRC, last time I looked only 4 statement/expression classes currently have some abbreviation defined.


Yep, that is what i'm seeing in this diff.

In D59214#1429384 <https://reviews.llvm.org/D59214#1429384>, @riccibruno wrote:

> It would probably be useful to have someone go systematically through the list of classes and fix this.


Yeah, i suspect that might be **really** beneficial.


Repository:
  rC Clang

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

https://reviews.llvm.org/D59214





More information about the cfe-commits mailing list