On Mon, Oct 18, 2021 at 11:57:19AM +0200, Richard Biener via Gcc-patches wrote:
> On Mon, Oct 18, 2021 at 6:25 AM Siddhesh Poyarekar <siddh...@gotplt.org>
> wrote:
> >
> > This is a minor cleanup to bail out early if the result of
> > __builtin_object_size is not assigned to anything and avoid initializing
> > the object size arrays.
>
> OK.
Yeah, fortunately we have expansion for __builtin_object_size so even if
the user tries hard to avoid DCE of the builtin with no LHS through
-fno-tree-dce etc., it shouldn't ICE but expand to -1, 0 or __builtin_trap ()
depending on the arguments it had.
> > gcc/ChangeLog:
> >
> > * tree-object-size (object_sizes_execute): Consolidate LHS null
> > check and do it early.
> >
> > Signed-off-by: Siddhesh Poyarekar <siddh...@gotplt.org>
Jakub
