[Lldb-commits] [PATCH] D62562: [Target] Introduce Process::GetLanguageRuntimes

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue May 28 20:37:17 PDT 2019


JDevlieghere added inline comments.


================
Comment at: source/Target/Thread.cpp:2216
+      GetProcess()->GetLanguageRuntimes();
+  for (LanguageRuntime *runtime : language_runtimes) {
+    if (!runtime)
----------------
if ` for (LanguageRuntime *runtime : GetProcess()->GetLanguageRuntimes()) {` fits on one line I would get rid of the temporary. 


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

https://reviews.llvm.org/D62562





More information about the lldb-commits mailing list