Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add output.testsuite to .gitignore since it isn't matched by output.testsuite.* #736

Merged
merged 1 commit into from
Apr 3, 2023
Merged

Add output.testsuite to .gitignore since it isn't matched by output.testsuite.* #736

merged 1 commit into from
Apr 3, 2023

Conversation

leekillough
Copy link
Collaborator

Add output.testsuite to .gitignore since it isn't matched by output.testsuite.*. I accidentally added output.testsuite in a commit with git add . after a test run.

@leekillough
Copy link
Collaborator Author

ping @fgvanzee

@fgvanzee
Copy link
Member

fgvanzee commented Apr 3, 2023

Thanks @leekillough!

@fgvanzee fgvanzee merged commit 3f1432a into flame:master Apr 3, 2023
leekillough added a commit to leekillough/blis that referenced this pull request Jun 7, 2023
Details:
- Added `output.testsuite` to .gitignore since it was previously not 
  being matched by `output.testsuite.*`.
ct-clmsn pushed a commit to ct-clmsn/blis that referenced this pull request Jul 29, 2023
Details:
- Added `output.testsuite` to .gitignore since it was previously not 
  being matched by `output.testsuite.*`.
fgvanzee added a commit that referenced this pull request May 22, 2024
- (cherry picked from 593d017)

CREDITS file update.

Details:
- Added attributions associated with commits:
  - 98d4678 9b1beec: @bartoldeman
  - 2b05948 059f151: @ct-clmsn
- Reordered attirubtion for @decandia50.
- (cherry picked from 259f684)

Optionally disable thread-local storage. (#735)

Details:
- Implemented a new configure option, --disable-tls, which allows the
  user to optionally disable the use of thread-local storage qualifiers
  on static variables in BLIS. This option will rarely be needed, but
  in some situations may allow BLIS to compile when TLS is unavailable.
  Thanks to Nick Knight for suggesting this option.
- Unlike the --disable-system option, --disable-tls does not forcibly
  disable threading. Instead, warnings of the possible consequences of
  using threading with TLS disabled are added to:
  - the output of './configure --help';
  - the output of 'configure' the --disable-tls option is parsed;
  - the informational header output by the testsuite.
  Thanks to Minh Quan Ho for suggesting these warnings.
- Modified frame/include/bli_lang_defs.h so that BLIS_THREAD_LOCAL is
  defined to nothing when BLIS_ENABLE_TLS is not defined.
- Defined bli_info_get_enable_tls(), which returns whether the cpp macro
  BLIS_ENABLE_TLS was defined.
- Edited --disable-system configure status output for clarity.
- Whitespace updates.
- (cherry picked from aea8e1d)

Add output.testsuite to .gitignore (#736)

Details:
- Added `output.testsuite` to .gitignore since it was previously not
  being matched by `output.testsuite.*`.
- (cherry picked from 3f1432a)

Added mm_algorithm pdf files (bp and pb).

Details:
- Added PDF versions of the PowerPoint files added in 17cd260.
- (cherry picked from 38fc523)

Added mm_algorithm pptx files (bp and pb).

Details:
- Added two PowerPoint files that contain slides depicting the classic
  Goto algorithm for matrix multiplication as well as its sister
  "panel-block" algorithm. These files reside in docs/diagrams.
- (cherry picked from 17cd260)

Move -fPIC insertion to subconfigs' make_defs.mk. (#738)

Details:
- Previously, common.mk was appending -fPIC to the CPICFLAGS variables
  set within the various subconfigurations' make_defs.mk files. This
  seemed somewhat unintuitive, and so now the -fPIC flag is assigned to
  the various subconfigs' CPICFLAGS variables in the respective
  make_defs.mk files.
- This also commit changes the logic in common.mk so that instead of
  appending, the variable is overwritten, but now *only* in the case
  of Windows (since apparently -fPIC needs to be omitted there). Thanks
  to Nick Knight for catching and reporting this weirdness.
- (cherry picked from 9d778e0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants