Skip to content

Update valgrind version build from source#38754

Closed
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/docker-update-valgrind
Closed

Update valgrind version build from source#38754
malfet wants to merge 1 commit intopytorch:masterfrom
malfet:malfet/docker-update-valgrind

Conversation

@malfet
Copy link
Copy Markdown
Contributor

@malfet malfet commented May 20, 2020

Why not use valgrind-3.15.0?
Also, build in in parallel (with -j4)

Also, build in in parallel (with -j4)
@malfet malfet requested review from ezyang, kostmo and seemethere May 20, 2020 02:05
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented May 20, 2020

💊 CI failures summary and remediations

As of commit be2bb2e (more details on the Dr. CI page):


  • 1/2 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)
  • 1/2 broken upstream at merge base 5af4e76 since May 19

🚧 1 ongoing upstream failure:

These were probably caused by upstream breakages that are not fixed yet:


ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker.

See how this bot performed.

This comment has been revised 2 times.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@malfet is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@malfet malfet deleted the malfet/docker-update-valgrind branch May 20, 2020 06:32
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@malfet merged this pull request in 1465970.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Why not use valgrind-3.15.0?
Also, build in in parallel (with -j4)
Pull Request resolved: pytorch#38754

Differential Revision: D21657357

Pulled By: malfet

fbshipit-source-id: 22b7761a6c9672477e32f16e56f58bdcce02a75c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants