[PATCH] D44048: [DebugInfo] Add DW_AT_byte_size to vectors

[Eric Christopher via Phabricator via llvm-commits]

echristo added inline comments.


================
Comment at: lib/CodeGen/AsmPrinter/DwarfUnit.cpp:1428
+  const auto Subrange = cast<DISubrange>(Elements[0]);
+  const auto CI = Subrange->getCount().dyn_cast<ConstantInt *>();
+  const int32_t NumVecElements = CI->getSExtValue();
----------------
probinson wrote:
> I think `get<ConstantInt *>` instead of `dyn_cast` so this will assert if it isn't the right type.
> 
Yes, because otherwise you could be grabbing a null pointer as well.


https://reviews.llvm.org/D44048