https://gcc.gnu.org/bugzilla/show_bug.cgi?id=121947

--- Comment #4 from Hongtao Liu <liuhongt at gcc dot gnu.org> ---
(In reply to H.J. Lu from comment #3)
> Created attachment 62385 [details]
> A patch
> 
> This is a test patch:
> 
> 1. Move pass_x86_cse after pass_split_all_insns.
> 2. Split 3 UNSPEC_INSN_FALSE_DEP patterns before RA.
> 
> Can we try if it works?

I'd rather keep the original implementation for below 2 reasons.

1) It needs quite a lot of work to do it as splitter way, and more efforts for
maintenance.

2) The tune is only for SPR(GLC), and no longer needed for the
successors(P-core) or any E-Core processors.

Reply via email to