Skip to content

modified:fftw/package.py,new file:a64fx-3.3.8.patch#18437

Closed
hirashima-ryota wants to merge 1 commit intospack:developfrom
hirashima-ryota:fftw-tune
Closed

modified:fftw/package.py,new file:a64fx-3.3.8.patch#18437
hirashima-ryota wants to merge 1 commit intospack:developfrom
hirashima-ryota:fftw-tune

Conversation

@hirashima-ryota
Copy link
Copy Markdown

Sorry for causing conflict.
fftw(v3.3.8) tunning version for a64fx(fj 4.2.0).
modified:fftw/package.py
new file:a64fx-3.3.8.patch

Copy link
Copy Markdown
Member

@alalazo alalazo left a comment

Choose a reason for hiding this comment

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

There are changes in this PR that undo previous PRs. Was that on purpose?

patch('pfft-3.3.4.patch', when="@3.3.4+pfft_patches", level=0)
patch('pgi-3.3.6-pl2.patch', when="@3.3.6-pl2%pgi", level=0)
patch('intel-configure.patch', when="@3:3.3.8%intel", level=0)
patch('a64fx-3.3.8.patch', when="@3.3.8 target=a64fx")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you provide a reference where this patch is discussed? Asking since the patch changes ~8k lines in the sources. Will it be accepted upstream for future releases?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

(See #18349 (comment) for the initial discussion on the question above)

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Sep 1, 2020

Duplicate of #18349

@alalazo alalazo marked this as a duplicate of #18349 Sep 1, 2020
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.

2 participants