[flang-commits] [PATCH] D104290: [flang] Check there's no dependency on C++ libs

[Jean Perier via Phabricator via flang-commits]

jeanPerier accepted this revision.
jeanPerier added a comment.
This revision is now accepted and ready to land.

Looks like a good idea to me to have this test, thanks !



================
Comment at: flang/test/lit.cfg.py:85
+    tools.append(ToolSubst('%libruntime',
+        command=os.path.join(config.flang_lib_dir, 'libFortranRuntime.a'),
+        unresolved='warn'))
----------------
I wonder if you should also add `libFortranDecimal.a` to this, `libFortranRuntime.a` depends upon it. The dependency is not required for your test, so maybe this could also be done when adding an I/O runtime call use that actually depends on it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104290