[PATCH] D123627: Correctly diagnose prototype redeclaration errors in C

[Eli Friedman via Phabricator via cfe-commits]

efriedma added a comment.

> The reason you get the weird behavior with the note pointing to the same line as the declaration is because rintf() is a predefined builtin: https://godbolt.org/z/j3W759M7a (note the same lovely diagnostic note behavior).

It points at the same location, but at least it says "'rintf' is a builtin" (diag::note_previous_builtin_declaration).  The new codepath specifically skips the code which emits that note.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123627