On Nov 3, 2016, at 2:29 AM, Senthil Kumar Selvaraj <senthil_kumar.selva...@atmel.com> wrote: > > The below patch adds a new effective target keyword (store_merge) for > use in the store_merging_xxx.c tests. > > The tests currently require non_strict_align, but they still fail for the > avr. > Eyeballing the dump, I found that the pass doesn't attempt merging as it is > unprofitable for a target like the avr which has only single byte > stores. > > I figured store merging is unlikely to be profitable for targets with > smallish word sizes, and added a check_effective_target_store_merge > that combines non_strict_align and int32plus. > > Is this ok for trunk?
Ok. If anyone knows of a reason why this would turn off these tests for a target that currently tests them and works ok, let us know.
