[flang-commits] [flang] [flang] use setsid to assign the child to prevent zombie as it will be clean up by init process (PR #77944)

Pete Steinfeld via flang-commits flang-commits at lists.llvm.org
Wed Jan 17 13:57:50 PST 2024


https://github.com/psteinfeld commented:

What's the purpose of this change?  Is there a test that's failing?

Also, I don't see the changes listed here, but when I try to build from this repository, I get errors.  The function `ExecuteCommandLine` declares a variable called `newCmd`.  The version of that declaration that I see in this repository looks like this:
```
    const char *newCmd{EnsureNullTerminated(
     ...
```
Then, later in this function this variable is deallocated.  For me, this causes the following error message:
```
/local/home/psteinfeld/main/yiwu/llvm-project/flang/runtime/execute.cpp:213:24: error: cast from type ‘const char*’ to type ‘void*’ casts away qualifiers [-Werror=cast-qual]
  213 |     FreeMemory((void *)newCmd);
      |                        ^~~~~~
At global scope:
cc1plus: error: unrecognized command line option ‘-Wno-ctad-maybe-unsupported’ [-Werror]
cc1plus: all warnings being treated as errors
```
```

https://github.com/llvm/llvm-project/pull/77944


More information about the flang-commits mailing list