From 3821ff03cd36c9f06664ba347ed4b484f1c697fe Mon Sep 17 00:00:00 2001 From: David Majnemer Date: Tue, 15 Jul 2014 23:01:10 +0000 Subject: [PATCH] X86: Simplify X86WindowsTargetObjectFile::getSectionForConstant There exists a helper function to abstract away the various differences between ConstantVector, ConstantDataVector, ConstantAggregateZero, etc. Use it to simplify X86WindowsTargetObjectFile::getSectionForConstant. llvm-svn: 213104 --- llvm/lib/Target/X86/X86TargetObjectFile.cpp | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/llvm/lib/Target/X86/X86TargetObjectFile.cpp b/llvm/lib/Target/X86/X86TargetObjectFile.cpp index 21ee047..f8bcd61 100644 --- a/llvm/lib/Target/X86/X86TargetObjectFile.cpp +++ b/llvm/lib/Target/X86/X86TargetObjectFile.cpp @@ -151,15 +151,9 @@ X86WindowsTargetObjectFile::getSectionForConstant(SectionKind Kind, uint64_t NumBits = VTy->getBitWidth(); if (NumBits == 128 || NumBits == 256) { COMDATSymName = NumBits == 128 ? "__xmm@" : "__ymm@"; - if (const auto *CDV = dyn_cast(C)) { - for (int I = CDV->getNumElements() - 1, E = -1; I != E; --I) - COMDATSymName += - scalarConstantToHexString(CDV->getElementAsConstant(I)); - } else { - const auto *CV = cast(C); - for (int I = CV->getNumOperands() - 1, E = -1; I != E; --I) - COMDATSymName += scalarConstantToHexString(CV->getOperand(I)); - } + for (int I = VTy->getNumElements() - 1, E = -1; I != E; --I) + COMDATSymName += + scalarConstantToHexString(C->getAggregateElement(I)); } } if (!COMDATSymName.empty()) { -- 2.7.4