[PATCH] D128830: [Pipelines] Introduce DAE after ArgumentPromotion

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin added inline comments.


================
Comment at: llvm/lib/Passes/PassBuilderPipelines.cpp:1622
       InlineContext{ThinOrFullLTOPhase::FullLTOPostLink,
-                          InlinePass::CGSCCInliner}));
+                    InlinePass::CGSCCInliner}));
 
----------------
psamolysov wrote:
> psamolysov wrote:
> > mtrofin wrote:
> > > same clang-format q
> > Should I revert the changes introduced by clang-format?
> If these formatting changes make sense, I can introduce them in a separate patch. Should I?
They are fine here, I was just checking. Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128830