[PATCH] D21737: [PATCH] [CodeGen] Insert TargetLibraryInfoWrapperPass before anything else.
Marcin KoĆcielnicki via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 25 03:17:20 PDT 2016
koriakin added inline comments.
================
Comment at: lib/CodeGen/BackendUtil.cpp:422
// Set up the per-function pass manager.
+ FPM.add(new TargetLibraryInfoWrapperPass(*TLII));
if (CodeGenOpts.VerifyModule)
----------------
mehdi_amini wrote:
> This seems unnecessary?
This ensures FPM passes get the right TLI as well - since I'm removing the PMBuilder.LIbraryInfo assignment, they would otherwise get the default-constructed ones. Or should I keep the assignment instead, and only special-case MPM?
Repository:
rL LLVM
https://reviews.llvm.org/D21737
More information about the cfe-commits
mailing list