It's better to change IPA_CG_ST_st_idx to a inline function.

2011/11/11 Yulei Sui <[email protected]>:
> Hello,
>
> There is an mapping alias tag bug when Creating NystromAliasAnalyzer for PU
> in IPA phase.
>
> curFilePUIdx() returns a UINT32 which is inconsistance of UINT64 for
> cg_st_idx.   All these leads to incorrect mapping to an alias tag
>
> The fixed patch is as follows:
>
>
> Index:
> osprey/be/com/nystrom_alias_analyzer.cxx
> ===================================================================
> --- osprey/be/com/nystrom_alias_analyzer.cxx    (revision 3797)
> +++ osprey/be/com/nystrom_alias_analyzer.cxx    (working copy)
> @@ -227,7 +227,7 @@
>    else {
>      cg = _constraintGraph;
>      if (ipaMode()) {
> -      cg_st_idx = IPA_CG_ST_st_idx(curFilePUIdx(), st);
> +      cg_st_idx = IPA_CG_ST_st_idx((UINT64)curFilePUIdx(), st);
>      }
>      else {
>        cg_st_idx = CG_ST_st_idx(st);
>
>
>
> Could a gate keeper check this?
>
> ------------------------------------------------------------------------------
> RSA(R) Conference 2012
> Save $700 by Nov 18
> Register now
> http://p.sf.net/sfu/rsa-sfdev2dev1
> _______________________________________________
> Open64-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/open64-devel
>
>



-- 
Regards,
Lai Jian-Xin

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
Open64-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/open64-devel

Reply via email to