Two minor comments:
1. Is it better to rename the functions like EBO_Is_FMA3 into TOP_Is_FMA3?
2. In EBO_Associate_FMA, there are indent issues in 3 lines:
+ // Now check for defs on the mul opnds
+ if (is_fma3) {
+ OP *mov_op;
All other changes look good to me.
2012/4/14 Berg, Michael <michael.b...@amd.com>
> Attached is a patch pertaining functionality for fma based
> disassociation and reassociation phased in standard passes before/after
> register allocation:****
>
> ****
>
> Can a gate keeper please view these changes.****
>
> ** **
>
> Thanks****
>
> ** **
>
> -Michael****
>
>
> ------------------------------------------------------------------------------
> For Developers, A Lot Can Happen In A Second.
> Boundary is the first to Know...and Tell You.
> Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
> http://p.sf.net/sfu/Boundary-d2dvs2
> _______________________________________________
> Open64-devel mailing list
> Open64-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/open64-devel
>
>
--
Regards,
Lai Jian-Xin
------------------------------------------------------------------------------
For Developers, A Lot Can Happen In A Second.
Boundary is the first to Know...and Tell You.
Monitor Your Applications in Ultra-Fine Resolution. Try it FREE!
http://p.sf.net/sfu/Boundary-d2dvs2
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel
