[PATCH] D107872: [SimplifyLibCalls] propagate tail flag on FORITIFY calls

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 10 16:36:25 PDT 2021


nickdesaulniers added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/SimplifyLibCalls.cpp:3041
+      if (CI->isTailCall() && !SimplifiedCI->isTailCall())
+        SimplifiedCI->setTailCall();
+
----------------
There's another call site of `FortifiedLibCallSimplifier.optimizeCall`. Perhaps it would be better to sink this check into that method? Thoughts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107872



More information about the llvm-commits mailing list