On Thu, Oct 7, 2021 at 9:47 AM Kito Cheng <kito.ch...@gmail.com> wrote: > > Hi Andrew: > > > Seems like this error should be an gcc_assert at this point. The > > error message should have come from expand_builtin___clear_cache > > already. > > Actually it will emit errors here, so put gcc_assert will cause ICE :p
The error message would have been emitted from expand_builtin___clear_cache and maybe_emit_call_builtin___clear_cache would not have been called from user code. All other uses of maybe_emit_call_builtin___clear_cache are internal to gcc and should have the correct mode so asserting is the right thing to do. Thanks, Andrew Pinski > > > And the check for VOIDmode should really be a check for CONST_INT. > > Yeah, that makes sense, thanks!
