On 11/04/13 06:27, Richard Biener wrote:
On Thu, Oct 31, 2013 at 10:02 AM, Ilya Enkovich <enkovich....@gmail.com> wrote:
Hi,
Here is a patch which hadles the problem with optimization of
BUILT_IN_CHKP_ARG_BND calls. Pointer Bounds Checker expects that argument of
this call is a default SSA_NAME of the PARM_DECL whose bounds we want to get.
The problem is in optimizations which may replace arg with it's copy or a known
value. This patch suppress such modifications.
This doesn't seem like a good fix. I suppose you require the same on
RTL, that is, have the incoming arg reg coalesced with the use?
In that case better set SSA_NAME_OCCURS_IN_ABNORMAL_PHI.
I agree. This seems like the wrong direction.
While I don't like abusing SSA_NAME_OCCURS_IN_ABNORMAL_PHI in this
manner, setting that flag should give them precisely the behaviour they
want. That makes me think SSA_NAME_OCCURS_IN_ABNORMAL_PHI is poorly
named. But addressing that is separate from this patch.
Ilya, can you convert your code to set SSA_NAME_OCCURS_IN_ABNORMAL_PHI
and drop this patch?
Jeff
