public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "rdapp at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/113249] RISC-V: regression testsuite errors -mtune=generic-ooo
Date: Tue, 09 Jan 2024 16:53:42 +0000	[thread overview]
Message-ID: <bug-113249-4-LRTcTKRaRk@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-113249-4@http.gcc.gnu.org/bugzilla/>

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

--- Comment #4 from Robin Dapp <rdapp at gcc dot gnu.org> ---
> One of the reasons I've been testing things with generic-ooo is because
> generic-ooo had initial vector pipelines defined. For cleaning up the
> scheduler, I copied over the generic-ooo pipelines into generic and sifive-7
> md files. As you mentioned, the scan dump fails are likely less optimal code
> sequences for the as a result of the cost model. I'm planning on sending up
> a patch in my series that adds -fno-schedule-insns -fno-schedule-insns2 to
> the dump scan tests that fail but do you think it would be better to hard
> code the tune instead?

It's a bit difficult to say, actually both is not ideal but there is no ideal
way anyway :)

Disabling scheduling is probably fine for all the intrinsics tests because it
can be argued that the expected output is very close to the input anyway.

For others it might depend on the intention of the test.  But, in order to get
them out of the way, I think it should be ok to just disabling scheduling and
take care of the intention of the test later.

  parent reply	other threads:[~2024-01-09 16:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-06  1:30 [Bug target/113249] New: " ewlu at rivosinc dot com
2024-01-08 19:29 ` [Bug target/113249] " rdapp at gcc dot gnu.org
2024-01-09  1:15 ` ewlu at rivosinc dot com
2024-01-09  1:19 ` ewlu at rivosinc dot com
2024-01-09 16:53 ` rdapp at gcc dot gnu.org [this message]
2024-02-01  1:33 ` cvs-commit at gcc dot gnu.org
2024-02-21 22:09 ` cvs-commit at gcc dot gnu.org
2024-02-21 22:10 ` cvs-commit at gcc dot gnu.org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-113249-4-LRTcTKRaRk@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).