r302777 - Reverted r302775
Serge Pavlov via cfe-commits
cfe-commits at lists.llvm.org
Thu May 11 01:46:52 PDT 2017
Sorry.
I have to investigate a fail in clang-tidy.
Thanks,
--Serge
2017-05-11 15:43 GMT+07:00 Richard Smith <richard at metafoo.co.uk>:
> When you revert a patch (even one of your own), please say why you
> reverted it in the commit message.
>
>
> On 11 May 2017 1:38 am, "Serge Pavlov via cfe-commits" <
> cfe-commits at lists.llvm.org> wrote:
>
> Author: sepavloff
> Date: Thu May 11 03:25:22 2017
> New Revision: 302777
>
> URL: http://llvm.org/viewvc/llvm-project?rev=302777&view=rev
> Log:
> Reverted r302775
>
> Added:
> cfe/trunk/test/Driver/arm-darwin-builtin.c
> - copied unchanged from r302774, cfe/trunk/test/Driver/arm-darw
> in-builtin.c
> Modified:
> cfe/trunk/lib/Driver/Driver.cpp
> cfe/trunk/test/Driver/aarch64-cpus.c
> cfe/trunk/test/Driver/amdgpu-features.c
> cfe/trunk/test/Driver/arm-default-build-attributes.s
> cfe/trunk/test/Driver/cl-outputs.c
> cfe/trunk/test/Driver/clang_f_opts.c
> cfe/trunk/test/Driver/cuda-external-tools.cu
> cfe/trunk/test/Driver/debug-options.c
> cfe/trunk/test/Driver/gfortran.f90
> cfe/trunk/test/Driver/split-debug.h
> cfe/trunk/test/Driver/unknown-arg.c
> cfe/trunk/test/Index/index-attrs.c
> cfe/trunk/test/Index/index-attrs.cpp
> cfe/trunk/tools/driver/driver.cpp
>
> Modified: cfe/trunk/lib/Driver/Driver.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Dri
> ver.cpp?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/lib/Driver/Driver.cpp (original)
> +++ cfe/trunk/lib/Driver/Driver.cpp Thu May 11 03:25:22 2017
> @@ -598,8 +598,6 @@ Compilation *Driver::BuildCompilation(Ar
> bool CCCPrintPhases;
>
> InputArgList Args = ParseArgStrings(ArgList.slice(1));
> - if (Diags.hasErrorOccurred())
> - return nullptr;
>
> // Silence driver warnings if requested
> Diags.setIgnoreAllWarnings(Args.hasArg(options::OPT_w));
>
> Modified: cfe/trunk/test/Driver/aarch64-cpus.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/aa
> rch64-cpus.c?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/aarch64-cpus.c (original)
> +++ cfe/trunk/test/Driver/aarch64-cpus.c Thu May 11 03:25:22 2017
> @@ -11,7 +11,7 @@
> // RUN: %clang -target arm64 -### -c %s 2>&1 | FileCheck
> -check-prefix=ARM64-GENERIC %s
> // RUN: %clang -target arm64 -mcpu=generic -### -c %s 2>&1 | FileCheck
> -check-prefix=ARM64-GENERIC %s
> // RUN: %clang -target arm64 -mlittle-endian -### -c %s 2>&1 | FileCheck
> -check-prefix=ARM64-GENERIC %s
> -// RUN: %clang -target arm64 -mlittle-endian -mcpu=generic -### -c %s
> 2>&1 | FileCheck -check-prefix=ARM64-GENERIC %s
> +// RUN: %clang -target arm64 -mlittle-endian -mcpu-generic -### -c %s
> 2>&1 | FileCheck -check-prefix=ARM64-GENERIC %s
>
> // ARM64-GENERIC: "-cc1"{{.*}} "-triple" "arm64{{.*}}" "-target-cpu"
> "generic"
>
>
> Modified: cfe/trunk/test/Driver/amdgpu-features.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/am
> dgpu-features.c?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/amdgpu-features.c (original)
> +++ cfe/trunk/test/Driver/amdgpu-features.c Thu May 11 03:25:22 2017
> @@ -1,7 +1,7 @@
> -// RUN: %clang -### -target amdgcn -x cl -S -emit-llvm -mcpu=kaveri
> -mamdgpu-debugger-abi=0.0 %s -o - 2>&1 \
> +// RUN: %clang -### -target amdgcn -x cl -S -emit-llvm -mcpu=kaveri
> -mamdgpu-debugger-abi=0.0 %s -o 2>&1 \
> // RUN: | FileCheck --check-prefix=CHECK-MAMDGPU-DEBUGGER-ABI-0-0 %s
> // CHECK-MAMDGPU-DEBUGGER-ABI-0-0: the clang compiler does not support
> '-mamdgpu-debugger-abi=0.0'
>
> -// RUN: %clang -### -target amdgcn -x cl -S -emit-llvm -mcpu=kaveri
> -mamdgpu-debugger-abi=1.0 %s -o - 2>&1 \
> +// RUN: %clang -### -target amdgcn -x cl -S -emit-llvm -mcpu=kaveri
> -mamdgpu-debugger-abi=1.0 %s -o 2>&1 \
> // RUN: | FileCheck --check-prefix=CHECK-MAMDGPU-DEBUGGER-ABI-1-0 %s
> // CHECK-MAMDGPU-DEBUGGER-ABI-1-0: "-target-feature"
> "+amdgpu-debugger-insert-nops" "-target-feature"
> "+amdgpu-debugger-reserve-regs" "-target-feature"
> "+amdgpu-debugger-emit-prologue"
>
> Modified: cfe/trunk/test/Driver/arm-default-build-attributes.s
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/ar
> m-default-build-attributes.s?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/arm-default-build-attributes.s (original)
> +++ cfe/trunk/test/Driver/arm-default-build-attributes.s Thu May 11
> 03:25:22 2017
> @@ -10,9 +10,9 @@
>
> // Option ignored C/C++ (since we always emit hardware and ABI build
> attributes
> // during codegen).
> -// RUN: %clang -target armv7--none-eabi -### -x c %s
> -mdefault-build-attributes 2>&1 \
> +// RUN: %clang -target armv7--none-eabi -### -x c %s
> -mdefault-build-attributes -verify 2>&1 \
> // RUN: | FileCheck %s -check-prefix CHECK-DISABLED
> -// RUN: %clang -target armv7--none-eabi -### -x c++ %s
> -mdefault-build-attributes 2>&1 \
> +// RUN: %clang -target armv7--none-eabi -### -x c++ %s
> -mdefault-build-attributes -verify 2>&1 \
> // RUN: | FileCheck %s -check-prefix CHECK-DISABLED
>
> // CHECK-DISABLED-NOT: "-arm-add-build-attributes"
>
> Modified: cfe/trunk/test/Driver/cl-outputs.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl
> -outputs.c?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/cl-outputs.c (original)
> +++ cfe/trunk/test/Driver/cl-outputs.c Thu May 11 03:25:22 2017
> @@ -73,7 +73,7 @@
> // RUN: %clang_cl /c /o .. -### -- %s 2>&1 | FileCheck
> -check-prefix=oCRAZY2 %s
> // oCRAZY2: "-o" "..obj"
>
> -// RUN: not %clang_cl /c %s -### /o 2>&1 | FileCheck
> -check-prefix=oMISSINGARG %s
> +// RUN: %clang_cl /c %s -### /o 2>&1 | FileCheck
> -check-prefix=oMISSINGARG %s
> // oMISSINGARG: error: argument to '/o' is missing (expected 1 value)
>
> // RUN: %clang_cl /c /omydir/ -### -- %s %s 2>&1 | FileCheck
> -check-prefix=CHECK-oMULTIPLESOURCEOK1 %s
> @@ -208,7 +208,7 @@
> // FeoDIRNAMEEXTDLL: "-out:foo.dir{{[/\\]+}}a.ext"
> // FeoDIRNAMEEXTDLL: "-implib:foo.dir{{[/\\]+}}a.lib"
>
> -// RUN: not %clang_cl -### /o 2>&1 | FileCheck
> -check-prefix=FeoMISSINGARG %s
> +// RUN: %clang_cl -### /o 2>&1 | FileCheck -check-prefix=FeoMISSINGARG %s
> // FeoMISSINGARG: error: argument to '/o' is missing (expected 1 value)
>
> // RUN: %clang_cl /ofoo /o bar -### -- %s 2>&1 | FileCheck
> -check-prefix=FeoOVERRIDE %s
>
> Modified: cfe/trunk/test/Driver/clang_f_opts.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cl
> ang_f_opts.c?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/clang_f_opts.c (original)
> +++ cfe/trunk/test/Driver/clang_f_opts.c Thu May 11 03:25:22 2017
> @@ -186,7 +186,7 @@
> // CHECK-NO-SLP-VECTORIZE-AGG-NOT: "-vectorize-slp-aggressive"
>
> // RUN: %clang -### -S -fextended-identifiers %s 2>&1 | FileCheck
> -check-prefix=CHECK-EXTENDED-IDENTIFIERS %s
> -// RUN: not %clang -### -S -fno-extended-identifiers %s 2>&1 | FileCheck
> -check-prefix=CHECK-NO-EXTENDED-IDENTIFIERS %s
> +// RUN: %clang -### -S -fno-extended-identifiers %s 2>&1 | FileCheck
> -check-prefix=CHECK-NO-EXTENDED-IDENTIFIERS %s
> // CHECK-EXTENDED-IDENTIFIERS: "-cc1"
> // CHECK-EXTENDED-IDENTIFIERS-NOT: "-fextended-identifiers"
> // CHECK-NO-EXTENDED-IDENTIFIERS: error: unsupported option
> '-fno-extended-identifiers'
>
> Modified: cfe/trunk/test/Driver/cuda-external-tools.cu
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/cu
> da-external-tools.cu?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/cuda-external-tools.cu (original)
> +++ cfe/trunk/test/Driver/cuda-external-tools.cu Thu May 11 03:25:22 2017
> @@ -24,8 +24,8 @@
> // RUN: | FileCheck -check-prefix ARCH64 -check-prefix SM20 -check-prefix
> DBG %s
>
> // --no-cuda-noopt-device-debug overrides --cuda-noopt-device-debug.
> -// RUN: %clang -### -target x86_64-linux-gnu --cuda-noopt-device-debug \
> -// RUN: --no-cuda-noopt-device-debug -O2 -c %s 2>&1 \
> +// RUN: %clang -### -target x86_64-linux-gnu --cuda-noopt-debug \
> +// RUN: --no-cuda-noopt-debug -O2 -c %s 2>&1 \
> // RUN: | FileCheck -check-prefix ARCH64 -check-prefix SM20 -check-prefix
> OPT2 %s
>
> // Regular compile without -O. This should result in us passing -O0 to
> ptxas.
>
> Modified: cfe/trunk/test/Driver/debug-options.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/de
> bug-options.c?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/debug-options.c (original)
> +++ cfe/trunk/test/Driver/debug-options.c Thu May 11 03:25:22 2017
> @@ -80,7 +80,7 @@
> // RUN: %clang -### -c -gdwarf-2 %s 2>&1 \
> // RUN: | FileCheck -check-prefix=G_ONLY_DWARF2 %s
> //
> -// RUN: not %clang -### -c -gfoo %s 2>&1 | FileCheck -check-prefix=G_ERR
> %s
> +// RUN: %clang -### -c -gfoo %s 2>&1 | FileCheck -check-prefix=G_NO %s
> // RUN: %clang -### -c -g -g0 %s 2>&1 | FileCheck -check-prefix=G_NO %s
> // RUN: %clang -### -c -ggdb0 %s 2>&1 | FileCheck -check-prefix=G_NO %s
> // RUN: %clang -### -c -glldb -g0 %s 2>&1 | FileCheck -check-prefix=G_NO
> %s
> @@ -171,8 +171,6 @@
> // G_PS4: "-dwarf-version=
> // G_PS4: "-generate-arange-section"
> //
> -// G_ERR: error: unknown argument:
> -//
> // G_NO: "-cc1"
> // G_NO-NOT: -debug-info-kind=
> //
>
> Modified: cfe/trunk/test/Driver/gfortran.f90
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/gf
> ortran.f90?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/gfortran.f90 (original)
> +++ cfe/trunk/test/Driver/gfortran.f90 Thu May 11 03:25:22 2017
> @@ -106,6 +106,7 @@
> ! RUN: -fsyntax-only \
> ! RUN: -funderscoring \
> ! RUN: -fwhole-file \
> +! RUN: -fworking-directory \
> ! RUN: -imultilib \
> ! RUN: -iprefix \
> ! RUN: -iquote \
> @@ -225,6 +226,7 @@
> ! CHECK: "-fstack-arrays"
> ! CHECK: "-funderscoring"
> ! CHECK: "-fwhole-file"
> +! CHECK: "-fworking-directory"
> ! CHECK: "-imultilib"
> ! CHECK: "-iprefix"
> ! CHECK: "-iquote"
>
> Modified: cfe/trunk/test/Driver/split-debug.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/sp
> lit-debug.h?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/split-debug.h (original)
> +++ cfe/trunk/test/Driver/split-debug.h Thu May 11 03:25:22 2017
> @@ -3,4 +3,13 @@
> // RUN: %clang -target x86_64-unknown-linux-gnu -gsplit-dwarf -c
> -fmodules -### %s 2> %t
> // RUN: FileCheck -check-prefix=CHECK-NO-ACTIONS < %t %s
> //
> +// RUN: %clang -target x86_64-unknown-linux-gnu -gsplit-dwarf -c
> -fmodules -emit-module -fmodules-embed-all-files -fno-implicit-modules
> -fno-implicit-module-maps -### %s 2> %t
> +// RUN: FileCheck -check-prefix=CHECK-NO-ACTIONS < %t %s
> +//
> +// FIXME: This should fail using clang, except that the type of the
> output for
> +// an object output with modules is given as clang::driver::types::TY_PCH
> +// rather than TY_Object.
> +// RUN: %clang -target x86_64-unknown-linux-gnu -gsplit-dwarf -c
> -fmodules -fmodule-format=obj -### %s 2> %t
> +// RUN: FileCheck -check-prefix=CHECK-NO-ACTIONS < %t %s
> +//
> // CHECK-NO-ACTIONS-NOT: objcopy
>
> Modified: cfe/trunk/test/Driver/unknown-arg.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/un
> known-arg.c?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Driver/unknown-arg.c (original)
> +++ cfe/trunk/test/Driver/unknown-arg.c Thu May 11 03:25:22 2017
> @@ -1,8 +1,8 @@
> -// RUN: not %clang %s -cake-is-lie -%0 -%d -HHHH
> -munknown-to-clang-option -print-stats -funknown-to-clang-option -### 2>&1
> | \
> +// RUN: %clang %s -cake-is-lie -%0 -%d -HHHH -munknown-to-clang-option
> -print-stats -funknown-to-clang-option -### 2>&1 | \
> // RUN: FileCheck %s
> // RUN: %clang_cl -cake-is-lie -%0 -%d -HHHH -munknown-to-clang-option
> -print-stats -funknown-to-clang-option -### -c -- %s 2>&1 | \
> // RUN: FileCheck %s --check-prefix=CL
> -// RUN: not %clang_cl -cake-is-lie -%0 -%d -HHHH
> -munknown-to-clang-option -print-stats -funknown-to-clang-option -c
> -Werror=unknown-argument -### -- %s 2>&1 | \
> +// RUN: %clang_cl -cake-is-lie -%0 -%d -HHHH -munknown-to-clang-option
> -print-stats -funknown-to-clang-option -c -Werror=unknown-argument -### --
> %s 2>&1 | \
> // RUN: FileCheck %s --check-prefix=CL-ERROR
> // RUN: %clang_cl -cake-is-lie -%0 -%d -HHHH -munknown-to-clang-option
> -print-stats -funknown-to-clang-option -c -Wno-unknown-argument -### -- %s
> 2>&1 | \
> // RUN: FileCheck %s --check-prefix=SILENT
>
> Modified: cfe/trunk/test/Index/index-attrs.c
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Index/ind
> ex-attrs.c?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Index/index-attrs.c (original)
> +++ cfe/trunk/test/Index/index-attrs.c Thu May 11 03:25:22 2017
> @@ -1,16 +1,16 @@
> -// RUN: c-index-test -index-file %s -target armv7-windows-gnu -fdeclspec
> 2>&1 | FileCheck %s
> +// RUN: c-index-test -index-file -check-prefix CHECK %s -target
> armv7-windows-gnu -fdeclspec
>
> void __declspec(dllexport) export_function(void) {}
> -// CHECK: [indexDeclaration]: kind: function | name: export_function |
> {{.*}} | lang: C
> +// CHECK: [indexDeclaraton]: kind: function | name: export_function |
> {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllexport)
> void __attribute__((dllexport)) export_gnu_attribute(void) {}
> -// CHECK: [indexDeclaration]: kind: function | name: export_gnu_attribute
> | {{.*}} | lang: C
> +// CHECK: [indexDeclaration] kind: function | name: export_gnu_attribute
> | {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllexport)
>
> void __declspec(dllimport) import_function(void);
> -// CHECK: [indexDeclaration]: kind: function | name: import_function |
> {{.*}} | lang: C
> +// CHECK: [indexDeclaration] kind: function | name: import_function |
> {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllimport)
> void __attribute__((dllimport)) import_gnu_attribute(void);
> -// CHECK: [indexDeclaration]: kind: function | name: import_gnu_attribute
> | {{.*}} | lang: C
> +// CHECK: [indexDeclaration] kind: function | name: import_gnu_function |
> {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllimport)
>
>
> Modified: cfe/trunk/test/Index/index-attrs.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Index/ind
> ex-attrs.cpp?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/test/Index/index-attrs.cpp (original)
> +++ cfe/trunk/test/Index/index-attrs.cpp Thu May 11 03:25:22 2017
> @@ -1,4 +1,4 @@
> -// RUN: c-index-test -index-file %s -target armv7-windows-gnu -fdeclspec
> | FileCheck %s
> +// RUN: c-index-test -index-file -check-prefix CHECK %s -target
> armv7-windows-gnu -fdeclspec
>
> struct __declspec(dllexport) export_s {
> void m();
> @@ -19,7 +19,7 @@ struct __declspec(dllimport) import_s {
> class __attribute__((dllexport)) export_gnu_s {
> void m();
> };
> -// CHECK: [indexDeclaration]: kind: c++-class | name: export_gnu_s |
> {{.*}} | lang: C++
> +// CHECK: [indexDeclaration]: kind: struct | name: export_gnu_s | {{.*}}
> | lang: C++
> // CHECK: <attribute>: attribute(dllexport)
> // CHECK: [indexDeclaration]: kind: c++-instance-method | name: m |
> {{.*}} | lang: C++
> // CHECK: <attribute>: attribute(dllexport)
> @@ -27,24 +27,24 @@ class __attribute__((dllexport)) export_
> class __attribute__((dllimport)) import_gnu_s {
> void m();
> };
> -// CHECK: [indexDeclaration]: kind: c++-class | name: import_gnu_s |
> {{.*}} | lang: C++
> +// CHECK: [indexDeclaration]: kind: struct | name: import_gnu_s | {{.*}}
> | lang: C++
> // CHECK: <attribute>: attribute(dllimport)
> // CHECK: [indexDeclaration]: kind: c++-instance-method | name: m |
> {{.*}} | lang: C++
> // CHECK: <attribute>: attribute(dllimport)
>
> extern "C" void __declspec(dllexport) export_function(void) {}
> -// CHECK: [indexDeclaration]: kind: function | name: export_function |
> {{.*}} | lang: C
> +// CHECK: [indexDeclaraton]: kind: function | name: export_function |
> {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllexport)
> extern "C" void __attribute__((dllexport)) export_gnu_function(void) {}
> -// CHECK: [indexDeclaration]: kind: function | name: export_gnu_function
> | {{.*}} | lang: C
> +// CHECK: [indexDeclaraton]: kind: function | name: export_gnu_function |
> {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllexport)
>
> extern "C" {
> void __declspec(dllimport) import_function(void);
> -// CHECK: [indexDeclaration]: kind: function | name: import_function |
> {{.*}} | lang: C
> +// CHECK: [indexDeclaration] kind: function | name: import_function |
> {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllimport)
> void __attribute__((dllimport)) import_gnu_function(void);
> -// CHECK: [indexDeclaration]: kind: function | name: import_gnu_function
> | {{.*}} | lang: C
> +// CHECK: [indexDeclaration] kind: function | name: import_gnu_function |
> {{.*}} | lang: C
> // CHECK: <attribute>: attribute(dllimport)
> }
>
>
> Modified: cfe/trunk/tools/driver/driver.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/driver/
> driver.cpp?rev=302777&r1=302776&r2=302777&view=diff
> ============================================================
> ==================
> --- cfe/trunk/tools/driver/driver.cpp (original)
> +++ cfe/trunk/tools/driver/driver.cpp Thu May 11 03:25:22 2017
> @@ -454,41 +454,40 @@ int main(int argc_, const char **argv_)
> SetBackdoorDriverOutputsFromEnvVars(TheDriver);
>
> std::unique_ptr<Compilation> C(TheDriver.BuildCompilation(argv));
> - int Res = 1;
> - if (C.get()) {
> - SmallVector<std::pair<int, const Command *>, 4> FailingCommands;
> + int Res = 0;
> + SmallVector<std::pair<int, const Command *>, 4> FailingCommands;
> + if (C.get())
> Res = TheDriver.ExecuteCompilation(*C, FailingCommands);
>
> - // Force a crash to test the diagnostics.
> - if (TheDriver.GenReproducer) {
> - Diags.Report(diag::err_drv_force_crash)
> + // Force a crash to test the diagnostics.
> + if (TheDriver.GenReproducer) {
> + Diags.Report(diag::err_drv_force_crash)
> << !::getenv("FORCE_CLANG_DIAGNOSTICS_CRASH");
>
> - // Pretend that every command failed.
> - FailingCommands.clear();
> - for (const auto &J : C->getJobs())
> - if (const Command *C = dyn_cast<Command>(&J))
> - FailingCommands.push_back(std::make_pair(-1, C));
> - }
> + // Pretend that every command failed.
> + FailingCommands.clear();
> + for (const auto &J : C->getJobs())
> + if (const Command *C = dyn_cast<Command>(&J))
> + FailingCommands.push_back(std::make_pair(-1, C));
> + }
>
> - for (const auto &P : FailingCommands) {
> - int CommandRes = P.first;
> - const Command *FailingCommand = P.second;
> - if (!Res)
> - Res = CommandRes;
> -
> - // If result status is < 0, then the driver command signalled an
> error.
> - // If result status is 70, then the driver command reported a fatal
> error.
> - // On Windows, abort will return an exit code of 3. In these cases,
> - // generate additional diagnostic information if possible.
> - bool DiagnoseCrash = CommandRes < 0 || CommandRes == 70;
> + for (const auto &P : FailingCommands) {
> + int CommandRes = P.first;
> + const Command *FailingCommand = P.second;
> + if (!Res)
> + Res = CommandRes;
> +
> + // If result status is < 0, then the driver command signalled an
> error.
> + // If result status is 70, then the driver command reported a fatal
> error.
> + // On Windows, abort will return an exit code of 3. In these cases,
> + // generate additional diagnostic information if possible.
> + bool DiagnoseCrash = CommandRes < 0 || CommandRes == 70;
> #ifdef LLVM_ON_WIN32
> - DiagnoseCrash |= CommandRes == 3;
> + DiagnoseCrash |= CommandRes == 3;
> #endif
> - if (DiagnoseCrash) {
> - TheDriver.generateCompilationDiagnostics(*C, *FailingCommand);
> - break;
> - }
> + if (DiagnoseCrash) {
> + TheDriver.generateCompilationDiagnostics(*C, *FailingCommand);
> + break;
> }
> }
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170511/c4dfac18/attachment-0001.html>
More information about the cfe-commits
mailing list