diff options
| author | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-06-13 19:55:04 +0000 |
|---|---|---|
| committer | Sanjoy Das <sanjoy@playingwithpointers.com> | 2016-06-13 19:55:04 +0000 |
| commit | d0bdf3e02ba61a704bae67f188ff7e5a8be135ed (patch) | |
| tree | 876b50fd5f819e9059bb48242e8990375ba3e533 /llvm/lib/Analysis/AliasAnalysis.cpp | |
| parent | 99646871a74669a33ccd4c004a2087e2a348ddd6 (diff) | |
| download | bcm5719-llvm-d0bdf3e02ba61a704bae67f188ff7e5a8be135ed.tar.gz bcm5719-llvm-d0bdf3e02ba61a704bae67f188ff7e5a8be135ed.zip | |
Fix AAResults::callCapturesBefore for operand bundles
Summary:
AAResults::callCapturesBefore would previously ignore operand
bundles. It was possible for a later instruction to miss its memory
dependency on a call site that would only access the pointer through a
bundle.
Patch by Oscar Blumberg!
Reviewers: sanjoy
Differential Revision: http://reviews.llvm.org/D21286
llvm-svn: 272580
Diffstat (limited to 'llvm/lib/Analysis/AliasAnalysis.cpp')
| -rw-r--r-- | llvm/lib/Analysis/AliasAnalysis.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Analysis/AliasAnalysis.cpp b/llvm/lib/Analysis/AliasAnalysis.cpp index 117f8cb123f..263dd48bfea 100644 --- a/llvm/lib/Analysis/AliasAnalysis.cpp +++ b/llvm/lib/Analysis/AliasAnalysis.cpp @@ -445,7 +445,7 @@ ModRefInfo AAResults::callCapturesBefore(const Instruction *I, unsigned ArgNo = 0; ModRefInfo R = MRI_NoModRef; - for (ImmutableCallSite::arg_iterator CI = CS.arg_begin(), CE = CS.arg_end(); + for (auto CI = CS.data_operands_begin(), CE = CS.data_operands_end(); CI != CE; ++CI, ++ArgNo) { // Only look at the no-capture or byval pointer arguments. If this // pointer were passed to arguments that were neither of these, then it |
