Changing some strange code into an assert; NFC
authorAaron Ballman <aaron@aaronballman.com>
Wed, 21 Sep 2022 18:51:17 +0000 (14:51 -0400)
committerAaron Ballman <aaron@aaronballman.com>
Wed, 21 Sep 2022 18:59:51 +0000 (14:59 -0400)
commit1ae5310ebbecadab52c1b7fc29596ac5b61795a3
treea862b54d4bbf5dbfc0813a33bebda2742a6f7e16
parentefd97c716b36ff266ffa1ec1b7a3c4a3f28917b9
Changing some strange code into an assert; NFC

This code was added in b65b1f322bd88513586a4539d2b5f18aeb698f3f, but it
was not noticed that the [[fallthrough]] behavior was very wrong. In C
mode, we would set the ParenExprType to CompoundLiteral and then
promptly overwrite that information by falling through.

After some investigation, I convinced myself that it is not possible to
hit this code path in C, only in C++. I've switched it to be an
assertion; I don't expect to hit it, but if we do hit it, that will at
least give us a code example we can use to reason about the intent of
the original code.
clang/lib/Parse/ParseExpr.cpp