[PATCH] D59467: [clang] Adding the Likely Attribute from C++2a to AST

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 22 06:40:54 PDT 2019


riccibruno added inline comments.


================
Comment at: clang/lib/Sema/SemaStmtAttr.cpp:74
+         ControlScope->getFlags() & Scope::FnTryCatchScope)
+      S.Diag(A.getLoc(), diag::err_must_appear_after_branch) << A.getName();
+  }
----------------
I think that you need to test for each of the above.


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

https://reviews.llvm.org/D59467





More information about the cfe-commits mailing list