[PATCH] D44884: Improving 'addArgumentAttrsFromCallsites'

Buddhika Chamith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 25 16:00:43 PDT 2018


chamibuddhika created this revision.
chamibuddhika added reviewers: nlopes, philames, echristo.
Herald added subscribers: llvm-commits, mgorny, jfb, mehdi_amini.

This patch aims to fix a TODO inside addArgumentAttrsFromCallsites method in function attribute inference code. This is more like a RFC patch where I need some feedback if the direction taken is valid. Idea was to introduce a PostDominatorTree argument to the method. It could be used to check if the callsite dominates the entry block and if so it is safe to transfer its callee attributes to the callers arguments.

This patch currently does *not* pass regression tests. I suspect this is due to an issue with how PostDominatorTree pass is introduced in the PassManager. I am still not completely familiar with how PassManager works.


Repository:
  rL LLVM

https://reviews.llvm.org/D44884

Files:
  include/llvm/Transforms/IPO/FunctionAttrs.h
  lib/Transforms/CMakeLists.txt
  lib/Transforms/IPO/FunctionAttrs.cpp
  lib/Transforms/IPO/PassManagerBuilder.cpp
  lib/Transforms/IPO/WholeProgramDevirt.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44884.139751.patch
Type: text/x-patch
Size: 33846 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180325/c1a2bcca/attachment.bin>


More information about the llvm-commits mailing list