diff options
author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-04-06 23:18:49 +0000 |
---|---|---|
committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-04-06 23:18:49 +0000 |
commit | 9d1cf4c1663ccd3c6b6168973215fad0f297ca35 (patch) | |
tree | e7900851342fcf73e633889a0a5344d3534598c1 /llvm/lib/Analysis/ModuleDebugInfoPrinter.cpp | |
parent | b6030b9dbf1762877795fabbea7c7b4b7dbf627e (diff) | |
download | llvm-9d1cf4c1663ccd3c6b6168973215fad0f297ca35.zip llvm-9d1cf4c1663ccd3c6b6168973215fad0f297ca35.tar.gz llvm-9d1cf4c1663ccd3c6b6168973215fad0f297ca35.tar.bz2 |
IR: Stop using DIDescriptor::is*() and auto-casting
`DIDescriptor`'s subclasses allow construction from incompatible
pointers, and `DIDescriptor` defines a series of `isa<>`-like functions
(e.g., `isCompileUnit()` instead of `isa<MDCompileUnit>()`) that clients
tend to use like this:
if (DICompileUnit(N).isCompileUnit())
foo(DICompileUnit(N));
These construction patterns work together to make `DIDescriptor` behave
differently from normal pointers.
Instead, use built-in `isa<>`, `dyn_cast<>`, etc., and only build
`DIDescriptor`s from pointers that are valid for their type.
I've split this into a few commits for different parts of LLVM and clang
(to decrease the patch size and increase the chance of review).
Generally the changes I made were NFC, but in a few places I made things
stricter if it made sense from the surrounded code.
Eventually a follow-up commit will remove the API for the "old" way.
llvm-svn: 234255
Diffstat (limited to 'llvm/lib/Analysis/ModuleDebugInfoPrinter.cpp')
0 files changed, 0 insertions, 0 deletions