diff options
| author | Dimitry Andric <dimitry@andric.com> | 2019-04-22 19:20:46 +0000 |
|---|---|---|
| committer | Dimitry Andric <dimitry@andric.com> | 2019-04-22 19:20:46 +0000 |
| commit | 87e7f895bb2cc1c3fabe9e6905f596091b2cbf5f (patch) | |
| tree | b21d4cec91a9f08bf756be2e6a32febba83a1903 /llvm/lib/Analysis/InstructionSimplify.cpp | |
| parent | 2b744665308fc8d30a3baecb4947f2bd81aa7d30 (diff) | |
| download | bcm5719-llvm-87e7f895bb2cc1c3fabe9e6905f596091b2cbf5f.tar.gz bcm5719-llvm-87e7f895bb2cc1c3fabe9e6905f596091b2cbf5f.zip | |
Use correct way to test for MIPS arch after rOMP355687
Summary:
I ran into some issues after rOMP355687, where __atomic_fetch_add was
being used incorrectly on x86, and this turns out to be caused by the
following added conditionals:
```
#if defined(KMP_ARCH_MIPS)
```
The problem is, these macros are always defined, and are either 0 or 1
depending on the architecture. E.g. the correct way to test for MIPS
is:
```
#if KMP_ARCH_MIPS
```
Reviewers: petarj, jlpeyton, Hahnfeld, AndreyChurbanov
Reviewed By: petarj, AndreyChurbanov
Subscribers: AndreyChurbanov, sdardis, arichardson, atanasyan, jfb, jdoerfert, openmp-commits, llvm-commits
Tags: #openmp
Differential Revision: https://reviews.llvm.org/D60938
llvm-svn: 358911
Diffstat (limited to 'llvm/lib/Analysis/InstructionSimplify.cpp')
0 files changed, 0 insertions, 0 deletions
