diff options
author | Tobias Grosser <tobias@grosser.es> | 2014-07-08 22:51:03 +0000 |
---|---|---|
committer | Tobias Grosser <tobias@grosser.es> | 2014-07-08 22:51:03 +0000 |
commit | ca7f76c4063a53d2d2837cf554394932f1edf7a0 (patch) | |
tree | ca9750f9d90ee6b672d6854e6e41772a18df66bf | |
parent | 926731b1e5a5e9240f818d371cc4259d2bed419a (diff) | |
download | bcm5719-llvm-ca7f76c4063a53d2d2837cf554394932f1edf7a0.tar.gz bcm5719-llvm-ca7f76c4063a53d2d2837cf554394932f1edf7a0.zip |
DominanceInfo is strongly preferred over RegionInfo
This is and always was strong community consensus. Make this clear in the header
in case newcomers may not be aware.
llvm-svn: 212570
-rw-r--r-- | llvm/include/llvm/Analysis/RegionInfo.h | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/llvm/include/llvm/Analysis/RegionInfo.h b/llvm/include/llvm/Analysis/RegionInfo.h index 82a788d1bb8..93a1a48720f 100644 --- a/llvm/include/llvm/Analysis/RegionInfo.h +++ b/llvm/include/llvm/Analysis/RegionInfo.h @@ -22,6 +22,16 @@ // itself is not, but in practice runtime seems to be in the order of magnitude // of dominance tree calculation. // +// WARNING: LLVM is generally very concerned about compile time such that +// the use of additional analysis passes in the default +// optimization sequence is avoided as much as possible. +// Specifically, if you do not need the RegionInfo, but dominance +// information could be sufficient please base your work only on +// the dominator tree. Most passes maintain it, such that using +// it has often near zero cost. In contrast RegionInfo is by +// default not available, is not maintained by existing +// transformations and there is no intention to do so. +// //===----------------------------------------------------------------------===// #ifndef LLVM_ANALYSIS_REGIONINFO_H |