[PATCH] D91083: [AsmPrinter] fix -disable-debug-info option

Hans Wennborg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 13 04:51:25 PST 2020


hans added a comment.

The test fails on Mac, see e.g. https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8863756117615904432/+/steps/package_clang/0/stdout

The output of the first run line is:

  $ bin/llc -disable-debug-info-print=true -exception-model=dwarf -o - ../llvm/test/CodeGen/Generic/disable-debug-info-print.ll
          .section        __TEXT,__text,regular,pure_instructions
          .build_version macos, 10, 15
          .globl  _main                           ## -- Begin function main
          .p2align        4, 0x90
  _main:                                  ## @main
  ## %bb.0:                               ## %entry
          xorl    %eax, %eax
          retq
                                          ## -- End function
          .globl  _helper                         ## -- Begin function helper
          .p2align        4, 0x90
  _helper:                                ## @helper
          .cfi_startproc
  ## %bb.0:                               ## %entry
          xorl    %eax, %eax
          retq
          .cfi_endproc
                                          ## -- End function
  .subsections_via_symbols

Note that the .file directive which the test checks for isn't there.

The failure is probably reproducible by passing a -mtriple=darwin something to llc.

I've reverted in 105ed27ed80dd47a9d32e72bbdd2a776a3318f38 <https://reviews.llvm.org/rG105ed27ed80dd47a9d32e72bbdd2a776a3318f38> in the meantime.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91083/new/

https://reviews.llvm.org/D91083



More information about the llvm-commits mailing list