Skip to content

Commit

Permalink
Rollup merge of #83442 - durin42:remove-questionable-macros, r=cuviper
Browse files Browse the repository at this point in the history
LLVMWrapper: attractive nuisance macros

This came up in the review of #83425: it's hard to imagine a use of
LLVM_VERSION_LE() or LLVM_VERSION_EQ() that's not asking for trouble
when a point release gets created, so let's just discard them to prevent
the issue.
  • Loading branch information
JohnTitor authored Mar 25, 2021
2 parents ee34453 + 04961d2 commit 67436c1
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions compiler/rustc_llvm/llvm-wrapper/LLVMWrapper.h
Original file line number Diff line number Diff line change
Expand Up @@ -33,13 +33,6 @@
(LLVM_VERSION_MAJOR > (major) || \
LLVM_VERSION_MAJOR == (major) && LLVM_VERSION_MINOR >= (minor))

#define LLVM_VERSION_EQ(major, minor) \
(LLVM_VERSION_MAJOR == (major) && LLVM_VERSION_MINOR == (minor))

#define LLVM_VERSION_LE(major, minor) \
(LLVM_VERSION_MAJOR < (major) || \
LLVM_VERSION_MAJOR == (major) && LLVM_VERSION_MINOR <= (minor))

#define LLVM_VERSION_LT(major, minor) (!LLVM_VERSION_GE((major), (minor)))

#include "llvm/IR/LegacyPassManager.h"
Expand Down

0 comments on commit 67436c1

Please sign in to comment.