diff options
| author | Hans Wennborg <hans@hanshq.net> | 2017-08-23 15:43:28 +0000 |
|---|---|---|
| committer | Hans Wennborg <hans@hanshq.net> | 2017-08-23 15:43:28 +0000 |
| commit | 66f6fc0a49231aee5a565822be46863ef6e4ed2d (patch) | |
| tree | 8280f342ab53171c634cd756f3b621a635fbfe96 /llvm/lib/Transforms/Scalar | |
| parent | b2c0794e307c69018e90d28ea602f628bf0b8e8d (diff) | |
| download | bcm5719-llvm-66f6fc0a49231aee5a565822be46863ef6e4ed2d.tar.gz bcm5719-llvm-66f6fc0a49231aee5a565822be46863ef6e4ed2d.zip | |
LowerAtomic: Don't skip optnone functions; atomic still need lowering (PR34020)
The lowering isn't really an optimization, so optnone shouldn't make a
difference. ARM relies on the pass running when using "-mthread-model
single", because in that mode, it doesn't run AtomicExpand. See bug for
more details.
Differential Revision: https://reviews.llvm.org/D37040
llvm-svn: 311565
Diffstat (limited to 'llvm/lib/Transforms/Scalar')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/LowerAtomic.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/LowerAtomic.cpp b/llvm/lib/Transforms/Scalar/LowerAtomic.cpp index 08e60b16bed..6f77c5bd0d0 100644 --- a/llvm/lib/Transforms/Scalar/LowerAtomic.cpp +++ b/llvm/lib/Transforms/Scalar/LowerAtomic.cpp @@ -155,8 +155,7 @@ public: } bool runOnFunction(Function &F) override { - if (skipFunction(F)) - return false; + // Don't skip optnone functions; atomics still need to be lowered. FunctionAnalysisManager DummyFAM; auto PA = Impl.run(F, DummyFAM); return !PA.areAllPreserved(); |
