[PATCH] D41207: Rename --icf-data and add a corresponding flag for functions

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

There exists -icf=all, -icf=safe and -icf=none to choose ICF levels, and this patch adds other flags (-ignore-{data,function}-address-equality) to control ICF levels. It doesn't seems like the new flags are in line with the existing options. I'd keep the existing notation which is -icf=<something>.

If we use clang as a precedence, one candidate would be -icf=everything (just like -Weverything as opposed to -Wall) which would do more than -icf=all.


https://reviews.llvm.org/D41207