[PATCH] D74825: SpeculativeExecution: fixed ingoring free execution

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D74825#1927949 <https://reviews.llvm.org/D74825#1927949>, @leonardchan wrote:

> In D74825#1927673 <https://reviews.llvm.org/D74825#1927673>, @arsenm wrote:
>
> > In D74825#1927641 <https://reviews.llvm.org/D74825#1927641>, @leonardchan wrote:
> >
> > > Is it expected for binary size increases to result from this? Between the commit for this patch and the commit before it, I'm seeing an increase in some fuchsia ZBIs by about 13 kB.
> >
> >
> > The pass isn't supposed to even run for CPU targets
>
>
> I think this pass is also running for non-GPU targets. I'm seeing this pass run on x86_64, aarch64, and riscv64 when building a toolchain for those targets.
>
> https://github.com/llvm/llvm-project/blob/a4cde9ad7b6f1a4cfef228f6cf2fc4911bf24c77/llvm/lib/Passes/PassBuilder.cpp#L436 seems to add it to the new PM default function pipeline that I think runs as long as optimizations are available.


That doesn't match what the comment says, or the old PM does (which does createSpeculativeExecutionIfHasBranchDivergencePassz)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74825