[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