awarzynski added a comment.

Nice!

In D126291#3555835 <https://reviews.llvm.org/D126291#3555835>, @mmuetzel wrote:

> Is this still a configuration error?

No :) Clearly the following `if` block from `tools::addFortranRuntimeLibs` is 
not entered:

  if (TC.getTriple().isKnownWindowsMSVCEnvironment()) {
      CmdArgs.push_back("Fortran_main.lib");
      CmdArgs.push_back("FortranRuntime.lib");
      CmdArgs.push_back("FortranDecimal.lib");
    } 

The missing symbols reported by the linker are from thes Fortran runtime 
libraries listed above. These libs should be included in the linker invocation 
generated by `flang-new`. I'm assuming that this makes sense - given the 
definition of isKnownWindowsMSVCEnvironment 
<https://github.com/llvm/llvm-project/blob/5fee1799f4d8da59c251e2d04172fc2f387cbe54/llvm/include/llvm/ADT/Triple.h#L576-L578>.
 If you use isOSWindows 
<https://github.com/llvm/llvm-project/blob/5fee1799f4d8da59c251e2d04172fc2f387cbe54/llvm/include/llvm/ADT/Triple.h#L571-L573>
 instead then it should work, right?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126291/new/

https://reviews.llvm.org/D126291

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to