[PATCH] D137242: [llvm-debuginfo-analyzer] Remove '--print=sizes' in test case.

Carlos Alberto Enciso via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 4 03:31:22 PDT 2022


CarlosAlbertoEnciso added a comment.

In D137242#3902403 <https://reviews.llvm.org/D137242#3902403>, @mgorny wrote:

> I suppose this makes sense as a stop-gap measure but the option should be fixed to work on all platforms.

You are correct. That option must work on all platforms.



================
Comment at: llvm/test/tools/llvm-debuginfo-analyzer/DWARF/06-dwarf-full-logical-view.test:8-15
+;  clang-armv7-vfpv3-2stage
+;  https://lab.llvm.org/buildbot/#/builders/182/builds/4232
+;
+;  clang-armv7-global-isel
+;  https://lab.llvm.org/buildbot/#/builders/186/builds/9519
+;
+;  clang-armv7-2stage
----------------
mgorny wrote:
> Buildbot logs expire after some time, so I don't think there's a value in linking them.
That is a good point. May keep only the buildbot link:

;  clang-armv7-vfpv3-2stage
;  https://lab.llvm.org/buildbot/#/builders/182
;
;  clang-armv7-global-isel
;  https://lab.llvm.org/buildbot/#/builders/186
;
;  clang-armv7-2stage
;  https://lab.llvm.org/buildbot/#/builders/187



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137242



More information about the llvm-commits mailing list