[PATCH] D115850: [LTO][codegen] Add TargetLibraryInfoWrapperPass initially

Freddy, Ye via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 15 23:01:58 PST 2021


FreddyYe added a comment.

CodeGenPasses is a legacy pass manager here. For now. `llc.cpp` has not switched to the new pass manager. `llc.cpp` does add this pass. And many passes in codegen need a right TargetLibraryInfo. If not adding it here with the module's triple info. TargetLibraryInfo will be initialized with a null string. That results in setting many library calls unavailable, as conservative as possible. I think it's not reasonable. Welcome discuss! Welcome adding more reviewers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115850



More information about the llvm-commits mailing list