6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-07-24 06:25:18 +00:00
Code Issues Projects Releases Wiki Activity

[REFACTOR] Push logic from MemDepPrinter into getNonLocalPointerDependency

Browse Source 
Previously, MemDepPrinter handled volatile and unordered accesses without involving MemoryDependencyAnalysis.  By making a slight tweak to the documented interface - which is respected by both callers - we can move this responsibility to MDA for the benefit of any future callers.  This is basically just cleanup.

In the future, we may decide to extend MDA's non local dependency analysis to return useful results for ordered or volatile loads.  I believe (but have not really checked in detail) that local dependency analyis does get useful results for ordered, but not volatile, loads.




git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@225483 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Philip Reames
2015-01-09 00:26:45 +00:00
parent d4ca466ed6
commit ce0e12f0b4
3 changed files with 30 additions and 38 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
include/llvm/Analysis/MemoryDependenceAnalysis.h
View File

@@ -366,9 +366,12 @@ namespace llvm {
/// getNonLocalPointerDependency - Perform a full dependency query for an
/// access to the QueryInst's specified (non-volatile) memory location,
/// returning the set of instructions that either define or clobber
/// the value.
/// access to the QueryInst's specified memory location, returning the set
/// of instructions that either define or clobber the value.
///
/// Warning: For a volatile query instruction, the dependencies will be
/// accurate, and thus usable for reordering, but it is never legal to
/// remove the query instruction.
///
/// This method assumes the pointer has a "NonLocal" dependency within
/// QueryInst's parent basic block.