Fix warning from r332654 with LLVM_ATTRIBUTE_USED

r332654 tried to fix an unused function warning with
a void cast. This approach worked for clang and gcc 
but not for MSVC. This commit replaces the void cast
with the LLVM_ATTRIBUTE_USED approach.

llvm-svn: 332910
This commit is contained in:
Diego Caballero 2018-05-21 22:12:38 +00:00
parent 9efb77e25f
commit 1bd5f2261d

View File

@ -26,7 +26,8 @@ static cl::opt<bool> EnableHCFGVerifier("vplan-verify-hcfg", cl::init(false),
/// Utility function that checks whether \p VPBlockVec has duplicate
/// VPBlockBases.
static bool hasDuplicates(const SmallVectorImpl<VPBlockBase *> &VPBlockVec) {
LLVM_ATTRIBUTE_USED static bool
hasDuplicates(const SmallVectorImpl<VPBlockBase *> &VPBlockVec) {
SmallDenseSet<const VPBlockBase *, 8> VPBlockSet;
for (const auto *Block : VPBlockVec) {
if (VPBlockSet.count(Block))
@ -52,7 +53,6 @@ static void verifyBlocksInRegion(const VPRegionBlock *Region) {
// TODO: This won't work for switch statements.
assert(!hasDuplicates(Successors) &&
"Multiple instances of the same successor.");
(void)hasDuplicates;
for (const VPBlockBase *Succ : Successors) {
// There must be a bi-directional link between block and successor.