diff options
| author | Matt Arsenault <Matthew.Arsenault@amd.com> | 2019-10-27 19:37:45 -0700 |
|---|---|---|
| committer | Matt Arsenault <arsenm2@gmail.com> | 2019-10-27 19:39:14 -0700 |
| commit | 9b0b626d2c19802bf3d56272cbc3a13d3c9ee5dd (patch) | |
| tree | ac397a23ba4a7dcfa66a50e0c83bfb3f763d657b | |
| parent | 40b0fa7ef2123866b2252ef6990040c2707cabe4 (diff) | |
| download | bcm5719-llvm-9b0b626d2c19802bf3d56272cbc3a13d3c9ee5dd.tar.gz bcm5719-llvm-9b0b626d2c19802bf3d56272cbc3a13d3c9ee5dd.zip | |
Use isConvergent helper instead of directly checking attribute
| -rw-r--r-- | llvm/lib/Transforms/Scalar/LoopUnswitch.cpp | 2 | ||||
| -rw-r--r-- | llvm/lib/Transforms/Scalar/Sink.cpp | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/LoopUnswitch.cpp b/llvm/lib/Transforms/Scalar/LoopUnswitch.cpp index b410df0c5f6..afd0249ac96 100644 --- a/llvm/lib/Transforms/Scalar/LoopUnswitch.cpp +++ b/llvm/lib/Transforms/Scalar/LoopUnswitch.cpp @@ -683,7 +683,7 @@ bool LoopUnswitch::processCurrentLoop() { for (auto &I : *BB) { auto CS = CallSite(&I); if (!CS) continue; - if (CS.hasFnAttr(Attribute::Convergent)) + if (CS.isConvergent()) return false; if (auto *II = dyn_cast<InvokeInst>(&I)) if (!II->getUnwindDest()->canSplitPredecessors()) diff --git a/llvm/lib/Transforms/Scalar/Sink.cpp b/llvm/lib/Transforms/Scalar/Sink.cpp index 90f3a2aa46e..72b586c85fe 100644 --- a/llvm/lib/Transforms/Scalar/Sink.cpp +++ b/llvm/lib/Transforms/Scalar/Sink.cpp @@ -78,7 +78,7 @@ static bool isSafeToMove(Instruction *Inst, AliasAnalysis &AA, if (auto *Call = dyn_cast<CallBase>(Inst)) { // Convergent operations cannot be made control-dependent on additional // values. - if (Call->hasFnAttr(Attribute::Convergent)) + if (Call->isConvergent()) return false; for (Instruction *S : Stores) |
