On 06/11/2013 09:44 AM, Argyrios Kyrtzidis wrote:
On Jun 10, 2013, at 10:34 PM, Tobias Grosser <[email protected]> wrote:
On 06/10/2013 05:36 PM, Argyrios Kyrtzidis wrote:
Author: akirtzidis
Date: Mon Jun 10 19:36:55 2013
New Revision: 183717
URL: http://llvm.org/viewvc/llvm-project?rev=183717&view=rev
Log:
[libclang] Allow building a precompiled preamble with compiler errors
A while ago we allowed libclang to build a PCH that had compiler errors; this
was to retain the performance
afforded by a PCH even if the user's code is in an intermediate state.
Extend this for the precompiled preamble as well.
rdar://14109828
Very nice. Thanks a lot!
- if (Diagnostics->hasErrorOccurred()) {
+ if (!Act->hasEmittedPreamblePCH()) {
// There were errors parsing the preamble, so no precompiled header was
// generated. Forget that we even tried.
// FIXME: Should we leave a note for ourselves to try again?
Just to be sure: Is this comment still up to date?
I've made it more accurate in r183751.
Great thanks.
I understand that we now keep the header in case of non-fatal errors,
but can not keep it if fatal errors occurred. Before, we bailed out even
in case of non-fatal errors.
This makes it definitely more clear.
Cheers,
Tobias
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits