-
Notifications
You must be signed in to change notification settings - Fork 15.2k
Closed
Labels
bugzillaIssues migrated from bugzillaIssues migrated from bugzilla
Description
| Bugzilla Link | 42251 |
| Resolution | FIXED |
| Resolved on | Jun 17, 2019 16:00 |
| Version | 8.0 |
| OS | Linux |
| Blocks | #40566 |
| Attachments | 0001-OpenMP-OMPT-Fix-locking-testcases-for-32-bit-archite.patch |
| Reporter | LLVM Bugzilla Contributor |
| CC | @jprotze,@tstellar |
| Fixed by commit(s) | r361158 r363628 |
Extended Description
The following commit is needed to fix openmp tests on i386. It doesn't apply cleanly, so I'm attaching a rebased version.
commit 4109d56
Author: Joachim Protze [email protected]
Date: Mon May 20 16:21:42 2019
[OpenMP][OMPT] Fix locking testcases for 32 bit architectures
https://reviews.llvm.org/D58454 did not fix the problem for a typical use
case of building LLVM with gcc or icc and then testing with the newly built
clang compiler.
The compilers do not agree on how to extend a 32-bit pointer to uint64, so
make the pointer unsigned first, before adjusting the size.
Patch by Joachim Protze
Differential Revision: https://reviews.llvm.org/D58506
llvm-svn: 361158
Metadata
Metadata
Assignees
Labels
bugzillaIssues migrated from bugzillaIssues migrated from bugzilla