diff options
| author | Ramkumar Ramachandra <artagnon@gmail.com> | 2015-02-09 23:02:10 +0000 |
|---|---|---|
| committer | Ramkumar Ramachandra <artagnon@gmail.com> | 2015-02-09 23:02:10 +0000 |
| commit | 3edf74fe29789044ce146f34ce6f79dfe390ab22 (patch) | |
| tree | 5af7a880fc059438f6601b823cd6bb38f8a4fa5b /llvm/lib/Transforms | |
| parent | 2e4b9e0a37a24025649cd046e4d9cdae09d0a77f (diff) | |
| download | bcm5719-llvm-3edf74fe29789044ce146f34ce6f79dfe390ab22.tar.gz bcm5719-llvm-3edf74fe29789044ce146f34ce6f79dfe390ab22.zip | |
[Statepoint] Improve two asserts, fix some style (NFC)
Summary:
It's important that our users immediately know what gc.safepoint_poll
is. Also fix the style of the declaration of CreateGCStatepoint, in
preparation for another change that will wrap it.
Reviewers: reames
Subscribers: llvm-commits
Differential Revision: http://reviews.llvm.org/D7517
llvm-svn: 228626
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp index 517c4c95aa7..7b10d543736 100644 --- a/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp +++ b/llvm/lib/Transforms/Scalar/PlaceSafepoints.cpp @@ -741,7 +741,8 @@ InsertSafepointPoll(DominatorTree &DT, Instruction *term, // different type inserted previously Function *F = dyn_cast<Function>(M->getOrInsertFunction("gc.safepoint_poll", ftype)); - assert(F && !F->empty() && "definition must exist"); + assert(F && "void @gc.safepoint_poll() must be defined"); + assert(!F->empty() && "gc.safepoint_poll must be a non-empty function"); CallInst *poll = CallInst::Create(F, "", term); // Record some information about the call site we're replacing |
