[all-commits] [llvm/llvm-project] b61a4a: [MC] Default MCContext::UseNamesOnTempLabels to fa...

Fangrui Song via All-commits all-commits at lists.llvm.org
Tue Feb 25 18:25:18 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: b61a4aaca5f8a1e71eeaee74d93cdd6fccbef065
      https://github.com/llvm/llvm-project/commit/b61a4aaca5f8a1e71eeaee74d93cdd6fccbef065
  Author: Fangrui Song <maskray at google.com>
  Date:   2020-02-25 (Tue, 25 Feb 2020)

  Changed paths:
    M llvm/include/llvm/MC/MCContext.h
    M llvm/lib/CodeGen/LLVMTargetMachine.cpp
    M llvm/lib/MC/MCAsmStreamer.cpp
    M llvm/tools/llvm-mc/llvm-mc.cpp
    M llvm/tools/llvm-ml/llvm-ml.cpp

  Log Message:
  -----------
  [MC] Default MCContext::UseNamesOnTempLabels to false and only set it to true for MCAsmStreamer

Only MCAsmStreamer (assembly output) needs to keep names of temporary labels created by
MCContext::createTempSymbol().

This change made the rL236642 optimization available for cc2as and
probably some other users.

This eliminates a behavior difference between llvm-mc -filetype=obj and cc1as, which caused
https://reviews.llvm.org/D74006#1890487

Reviewed By: efriedma

Differential Revision: https://reviews.llvm.org/D75097




More information about the All-commits mailing list