[all-commits] [llvm/llvm-project] 8b725f: Comment and adjust style in the newly introduced M...

Philip Reames via All-commits all-commits at lists.llvm.org
Fri Dec 20 12:04:13 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 8b725f0459eee468ed7f9935fba3278fcb4997b1
      https://github.com/llvm/llvm-project/commit/8b725f0459eee468ed7f9935fba3278fcb4997b1
  Author: Philip Reames <listmail at philipreames.com>
  Date:   2019-12-20 (Fri, 20 Dec 2019)

  Changed paths:
    M llvm/include/llvm/MC/MCAsmBackend.h
    M llvm/include/llvm/MC/MCFragment.h

  Log Message:
  -----------
  Comment and adjust style in the newly introduced MCBoundaryAlignFragment infrastructure.  More to follow.


  Commit: 4024d49edc1598a6f8017df541147b38bf1e2818
      https://github.com/llvm/llvm-project/commit/4024d49edc1598a6f8017df541147b38bf1e2818
  Author: Philip Reames <listmail at philipreames.com>
  Date:   2019-12-20 (Fri, 20 Dec 2019)

  Changed paths:
    M llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp

  Log Message:
  -----------
  Fix a memory leak introduced w/the instruction padding support in rG14fc20ca6282

Should have caught this in review, but only noticed when addressing post commit style items.  We were creating a new instance of the X86MCInstrInfo class, and then never reclaiming the memory.  This wasn't even conditional on the new off by default flags, so it was an unconditional leak.


Compare: https://github.com/llvm/llvm-project/compare/fe17b30a7957...4024d49edc15


More information about the All-commits mailing list