[PATCH] D120051: [flang][driver] Make `flang-new` always generate run-time type info

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 17 07:09:33 PST 2022


awarzynski added inline comments.


================
Comment at: flang/lib/Frontend/FrontendAction.cpp:194
+  }
+
+  return true;
----------------
jeanPerier wrote:
> I wonder if you should not always complain here if `!ci.getRtTyTables().schemata`. Currently, it seems `DebugDumpSymbolsAction` and `DebugDumpSymbolsAction` would complain. I think at least the regular
> compiler workflows should complain if it was not able to generate those. Better than to wait for more cryptic link time errors.
Good point - this would also mean better code re-use. Let me update!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120051



More information about the llvm-commits mailing list