[PATCH 2/2] R600: Remove some redundant initializations from AMDGPUMCAsmInfo

Matt Arsenault Matthew.Arsenault at amd.com
Mon Oct 6 15:42:52 PDT 2014


On 10/06/2014 12:28 PM, Tom Stellard wrote:
> ---
>   lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp | 10 ----------
>   1 file changed, 10 deletions(-)
>
> diff --git a/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp b/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp
> index a795f3d..71bad75 100644
> --- a/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp
> +++ b/lib/Target/R600/MCTargetDesc/AMDGPUMCAsmInfo.cpp
> @@ -15,17 +15,12 @@ AMDGPUMCAsmInfo::AMDGPUMCAsmInfo(StringRef &TT) : MCAsmInfoELF() {
>     HasSingleParameterDotFile = false;
>     //===------------------------------------------------------------------===//
>     HasSubsectionsViaSymbols = true;
> -  HasMachoZeroFillDirective = false;
> -  HasMachoTBSSDirective = false;
> -  HasStaticCtorDtorReferenceInStaticMode = false;
>     LinkerRequiresNonEmptyDwarfLines = true;
>     MaxInstLength = 16;
>     SeparatorString = "\n";
>     CommentString = ";";
> -  LabelSuffix = ":";
>     InlineAsmStart = ";#ASMSTART";
>     InlineAsmEnd = ";#ASMEND";
> -  AssemblerDialect = 0;
>   
>     //===--- Data Emission Directives -------------------------------------===//
>     ZeroDirective = ".zero";
> @@ -35,14 +30,9 @@ AMDGPUMCAsmInfo::AMDGPUMCAsmInfo(StringRef &TT) : MCAsmInfoELF() {
>     Data16bitsDirective = ".short\t";
>     Data32bitsDirective = ".long\t";
>     Data64bitsDirective = ".quad\t";
> -  GPRel32Directive = nullptr;
>     SunStyleELFSectionSwitchSyntax = true;
>     UsesELFSectionDirectiveForBSS = true;
>   
> -  //===--- Alignment Information ----------------------------------------===//
> -  AlignmentIsInBytes = true;
> -  TextAlignFillValue = 0;
> -
>     //===--- Global Variable Emission Directives --------------------------===//
>     GlobalDirective = ".global";
>     HasSetDirective = false;
LGTM



More information about the llvm-commits mailing list