[PATCH] D81244: [StackSafety] Control paramer access summary from frontend

Evgenii Stepanov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 11 14:53:43 PDT 2020


eugenis added a comment.

I like this in principle. Makes a lot more sense to control this directly from clang than to rely on function attributes.



================
Comment at: llvm/lib/Bitcode/Writer/BitcodeWriterPass.cpp:31
+                       EmitModuleHash);
+  }
   return PreservedAnalyses::all();
----------------
What does this chunk do?


================
Comment at: llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:456
+    function_ref<AAResults &(Function &)> AARGetter, Module &M,
+    function_ref<ModuleSummaryIndex()> IndexGetter,
+    function_ref<const StackSafetyInfo *(const Function &)> SSIGetter) {
----------------
Why do you need to turn ModuleSummaryIndex into a getter?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81244





More information about the cfe-commits mailing list