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

Reply via email to