diff options
| author | Craig Topper <craig.topper@gmail.com> | 2017-03-27 02:38:17 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@gmail.com> | 2017-03-27 02:38:17 +0000 |
| commit | 74fb7ac2e081e70ddfa2237dd84cefd680573792 (patch) | |
| tree | b945e7e8cd1f96fb36fd9a15126762a398a46f09 /llvm/lib/IR/Module.cpp | |
| parent | 224b19d626b2c8ac81499bb5c4f2d25f49d2d002 (diff) | |
| download | llvm-74fb7ac2e081e70ddfa2237dd84cefd680573792.zip llvm-74fb7ac2e081e70ddfa2237dd84cefd680573792.tar.gz llvm-74fb7ac2e081e70ddfa2237dd84cefd680573792.tar.bz2 | |
[IR] Share implementation of pairs of const and non-const methods in BasicBlock using the const version instead of the non-const version
Summary:
During post-commit review of a previous change I made it was pointed out that const casting 'this' is technically a bad practice. This patch re-implements all of the methods in BasicBlock that do this to use the const BasicBlock version and const_cast the return value instead.
I think there are still many other classes that do similar things. I may look at more in the future.
Reviewers: dblaikie
Reviewed By: dblaikie
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D31377
llvm-svn: 298827
Diffstat (limited to 'llvm/lib/IR/Module.cpp')
0 files changed, 0 insertions, 0 deletions