[PATCH] D23885: [ThinLTO] add constArgumentsBitmask to caller summary

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 25 14:06:44 PDT 2016


Prazek added a comment.

In https://reviews.llvm.org/D23885#525493, @mehdi_amini wrote:

> Why a bit mask and not an integer with the number of arguments that are constant? 
>  Do you plan to benefit from knowing which argument is constant?


Yes, in order to make better heuristic, like the one that Easwaran come up with

- have some bitmask to say if the argument is compered inside function body as heuristic to determine if branch depends on it.

I will need the the information about the arguments.


https://reviews.llvm.org/D23885





More information about the llvm-commits mailing list