[clang] [clang-tools-extra] [lldb] [clang][TypePrinter][NFC] Turn SuppressTagKeyword into an enum (PR #171160)

Matheus Izvekov via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 8 09:33:25 PST 2025


mizvekov wrote:

As I pointed out in the original patch, maybe there was a confusion on my part where I assumed we would have a three valued enum, but the direction actually would make it four valued, and these would be controlling the suppression of the keyword in orthogonal places.

Can you clarify?

I don't mind bool vs two valued enum, if that's your preference, but the tying up of two separate conditions into one variable is making the other patch look awkward.

https://github.com/llvm/llvm-project/pull/171160


More information about the cfe-commits mailing list