[PATCH] D109775: [FuncSpec] Fix isConstant check for globals

Thu Sep 16 01:47:06 PDT 2021

ChuanqiXu added a comment.

In D109775#3003332 <https://reviews.llvm.org/D109775#3003332>, @SjoerdMeijer wrote:

> I have precommitted the test in rGa4e437e3c959 <https://reviews.llvm.org/rGa4e437e3c959ac0cb2edb733d081edc95a4fff22>.
>
> Specialising on the address confused me, i.e. I had not yet consciously thought about allowing/disallowing it. But now having looked a bit more into this, I don't see the problem, and I don't see why we would disallow it with this patch. So think I will abandon this patch. Any thoughts?

I prefer to disallowing specializing on the address of variable globals. Since it looks like not beneficial in most cases. And if we could be sure that it wouldn't be beneficial, I think we could disallow such cases.

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

https://reviews.llvm.org/D109775