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

--- Comment #18 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Kito Cheng from comment #17)
> Created attachment 47920 [details]
> update-local-align-pass.patch
> 
> Hi Jakub:
> 
> I got your point, and I agree with your point, estimate_stack_frame_size not
> the right place to update alignment.
> 
> I write a patch to add a new pass to update the data alignment, executed at
> begging of pass_all_early_optimizations, PoC patch attached, it's not fully
> tested yet, how do you think about this approach? 

Could you change pass_data_ipa_increase_alignment instead?  Which does a
similar thing except for global variables.

Reply via email to