[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    [PATCH] D53953: [clang-tidy] Get ClangTidyContext out of the business of storing diagnostics. NFC
From:       Alexander Kornienko via Phabricator via cfe-commits <cfe-commits () lists ! llvm ! org>
Date:       2018-10-31 22:52:30
Message-ID: ce2a961807ebd82c8c5cfebd6ec8a82e () localhost ! localdomain
[Download RAW message or body]

alexfh added a comment.

> error: unable to handle compilation, expected exactly one compiler job in '' \
> [clang-diagnostic-error] Suppressed 1 warnings (1 with check filters).
> Found compiler error(s).

That's preferred to the current behavior, but from a user perspective a more \
descriptive message would be better (like "can't find file /some/file" or "can't open \
file /some/file"). I don't know whether it's better to do in clang proper or in \
clang-tidy, but it can be handled separately from this patch.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53953



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic