[PATCH] D111570: [clang/CFG] Don't explicitly add AttributedStmtClass to AlwaysAddList

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 12 05:42:22 PDT 2021


thakis added a comment.

Yes, that's my understanding. (I'm not super familiar with this code either.)

The line was added in the change that added -Wimplicit-fallthrough (84837d5b5aa0d), and all the -Wimplicit-fallthrough tests still pass with this. Chances are it was unnecessary back then too.

(`addStmt` is from ea9fcffe035d, way older.)


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

https://reviews.llvm.org/D111570



More information about the cfe-commits mailing list