[PATCH][Win] Fix assertion failure when passing 'nul' in input to cc1.

Andrea Di Biagio andrea.dibiagio at gmail.com
Tue Apr 29 12:30:25 PDT 2014


Hi Rafael,

On Tue, Apr 29, 2014 at 7:21 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> +  #else
>
> +    file_status(file_type Type) : Type(Type) {}
>
> This is never used, right? We always have windows or unix.

Good point.
That #else could be removed.
I noticed however that in "llvm/Support/Process.h", if the OS is
neither Windows nor Linux, we #error with "Unsupported operating
system.".
Should I add #error in case of unsupported OS or do you prefer that I
remove the #else entirely?

>
>
> While you are at it, would you mind also updating the default constructor?

Sure, I will fix the default constructor.

>
>
> Do we also crash in some llvm tools like llvm-as? That would allow us
> to test directly from llvm.

Unfortunately I am only able to reproduce the assertion failure with clang..
I just finished testing llvm-as, llvm-dis, llc and opt; none of those
tools crash for me.

Andrea
>
> LGTM with those changes.
>
> Cheers,
> Rafael




More information about the cfe-commits mailing list