[clang] [clang] Sema::isSimpleTypeSpecifier return true for _Bool in c99 (PR #72204)

Carl Peto via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 17 18:18:57 PST 2023


carlos4242 wrote:

The dup  in Format seems to have already drifted from the one in Sema so putting them together might be a significant change, a bit more than I'm comfortable taking on... I don't want to break or alter behaviour for someone's formatter just to fix a tiny issue I found in a backwater part of Sema. :)

I'm thinking this probably isn't a testable patch TBH. I came across this issue due to the swift compiler linking into libClang and calling this function. Within clang itself, it looks like it's only used for some obscure objective C case and an equally obscure error case. The only "test" I can envisage is building some kind of artificial stand-alone command line tool that links into libClang and tests it, doesn't seem like a sensible use of anyone's CPU for such a straight bufix?

---

But it's a decision for the maintainer....?

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


More information about the cfe-commits mailing list