[Beignet] [PATCH 5/5] GBE: implement further phi mov optimization based on intra-BB interefering analysis.

Song, Ruiling ruiling.song at intel.com
Mon Sep 21 20:54:50 PDT 2015


I just think of another optimization opportunity that may be missed in your algorithm.
As you use a map<phiCopySrc, phiCopy> to record the possible to-be-coaleased pair.
The phiCopySrc may be used in another phiNode in the same way.
Which the algorithm would not record. We may do it later.
Could you inline related comment into the patch?
Then others could easily understand the code.
Anyway, the patchset looks good.

Thanks!
Ruiling


More information about the Beignet mailing list