[all-commits] [llvm/llvm-project] b605cf: [AsmParser][SystemZ][z/OS] Reland "Introduce HLASM...

Anirudh Prasad via All-commits all-commits at lists.llvm.org
Tue Mar 16 14:12:09 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b605cfb336989705f391d255b7628062d3dfe9c3
      https://github.com/llvm/llvm-project/commit/b605cfb336989705f391d255b7628062d3dfe9c3
  Author: Anirudh Prasad <anirudh_prasad at hotmail.com>
  Date:   2021-03-16 (Tue, 16 Mar 2021)

  Changed paths:
    M llvm/include/llvm/MC/MCAsmInfo.h
    M llvm/lib/MC/MCParser/AsmLexer.cpp
    M llvm/lib/Target/SystemZ/MCTargetDesc/SystemZMCAsmInfo.cpp
    A llvm/unittests/MC/SystemZ/CMakeLists.txt
    A llvm/unittests/MC/SystemZ/SystemZAsmLexerTest.cpp

  Log Message:
  -----------
  [AsmParser][SystemZ][z/OS] Reland "Introduce HLASM Comment Syntax"

- Previously, https://reviews.llvm.org/D97703 was [[ https://reviews.llvm.org/D98543 | reverted ]] as it broke when building the unit tests when shared libs on.
- This patch reverts the "revert" and makes two minor changes
- The first is it also links in the MCParser lib when building the unittest. This should resolve the issue when building with with shared libs on and off
- The second renames the name of the unit test from `SystemZAsmLexer` to `SystemZAsmLexerTests` since the convention for unittest binaries is to suffix the name of the unit test with "Tests"

Reviewed By: Kai

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




More information about the All-commits mailing list