public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug middle-end/93086] build_nonstandard_integer_type should take signop instead of int
       [not found] <bug-93086-4@http.gcc.gnu.org/bugzilla/>
@ 2020-05-04 23:57 ` pinskia at gcc dot gnu.org
  2021-07-13  3:41 ` pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 2+ messages in thread
From: pinskia at gcc dot gnu.org @ 2020-05-04 23:57 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
I have a patch, in some places I am able to remove some extra varibles, in
others I need to add one.  There was an mix use of 0/1 and false/true even in
the same file for the argument of build_nonstandard_integer_type.  This new
argument type makes it more explicit what is wanted there.

I did not do signed_or_unsigned_type_for yet either; I will have one after I
test my current patch; this part will require much more changes to the
front-end as I am going to change the language hook too.

Note there are a few more places where signop; especially in gimple to rtl
expansion code.

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug middle-end/93086] build_nonstandard_integer_type should take signop instead of int
       [not found] <bug-93086-4@http.gcc.gnu.org/bugzilla/>
  2020-05-04 23:57 ` [Bug middle-end/93086] build_nonstandard_integer_type should take signop instead of int pinskia at gcc dot gnu.org
@ 2021-07-13  3:41 ` pinskia at gcc dot gnu.org
  1 sibling, 0 replies; 2+ messages in thread
From: pinskia at gcc dot gnu.org @ 2021-07-13  3:41 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #3 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Created attachment 51145
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=51145&action=edit
Patch in test

This patch is bigger than I wanted it to be as there has been many more uses of
build_nonstandard_integer_type than when I originally looked into it.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-07-13  3:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-93086-4@http.gcc.gnu.org/bugzilla/>
2020-05-04 23:57 ` [Bug middle-end/93086] build_nonstandard_integer_type should take signop instead of int pinskia at gcc dot gnu.org
2021-07-13  3:41 ` pinskia at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).