[PATCH] D143624: Inlining: Run the legacy AlwaysInliner before the regular inliner.

Arthur Eubanks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 9 10:47:18 PST 2023


aeubanks added a comment.

__clang_hip_math.hip is annoying...

We'll need to remove the `MandatoryFirst` inliner in `ModuleInlinerWrapperPass`, although not sure if @mtrofin has any issues with that or not

This isn't quite what I had initially thought, but this might be better. (I was thinking that we sort the calls in the inliner to visit alwaysinline calls first, but that might cause more compile time issues since we have to update the call graph after visiting all the calls in a function, but we might be visiting every function twice if we first batch process the alwaysinline calls then all other calls)



================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1082
 
+  MPM.addPass(AlwaysInlinerPass(/*InsertLifetimeIntrinsics=*/false));
+
----------------
I think we want to insert lifetime intrinsics when optimizing


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143624



More information about the cfe-commits mailing list