[all-commits] [llvm/llvm-project] 953bb5: [test] Make sure the test program in GetErrcMessag...

zero9178 via All-commits all-commits at lists.llvm.org
Tue Mar 16 04:23:03 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 953bb5e5c8f60dc769942a3615d800fe166ffd1d
      https://github.com/llvm/llvm-project/commit/953bb5e5c8f60dc769942a3615d800fe166ffd1d
  Author: Markus Böck <markus.boeck02 at gmail.com>
  Date:   2021-03-16 (Tue, 16 Mar 2021)

  Changed paths:
    M llvm/cmake/modules/GetErrcMessages.cmake

  Log Message:
  -----------
  [test] Make sure the test program in GetErrcMessages.cmake exits normally.

If for some reason the test program does not exit normally it'd currently lead to a false positive and it's stdout output being assigned to the output variable.

Instead, check the test program exited normally before assigning the process output to the out variable.

Follow up on rGaf2796c76d2ff4b73165ed47959afd35a769beee
Fixes an issue discovered post commit in https://reviews.llvm.org/D98278




More information about the All-commits mailing list