[PATCH] D151774: [LICM] Don't hoist threadlocals within presplit coroutines

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 5 22:18:06 PDT 2023


ChuanqiXu marked an inline comment as done.
ChuanqiXu added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:1228
+    if (CI->onlyReadsMemory() && CI->getFunction()->isPresplitCoroutine())
+      return false;
+
----------------
nikic wrote:
> Can you please move this to the existing Behavior check below?
Done. Sorry for not noticing it.


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

https://reviews.llvm.org/D151774



More information about the llvm-commits mailing list