[PATCH] D98559: [lld-macho] Implement -dependency_info (partially - more opcodes needed)

Vy Nguyen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 23 09:54:52 PDT 2021


oontvoo updated this revision to Diff 332708.
oontvoo added a comment.

Updated diff. rebase.

So it's turned out that this was likely a clang bug (or unimplemented feature).
In clang older than 10.x, it didn't know to cast the enum (DepOpCode) to its underlying type. Hence,  `<< DepOpCode` was ambiguous to it because the enum could be converted to any of the integer types.

Simple repro: https://gcc.godbolt.org/z/bT6zE7x77


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98559

Files:
  lld/MachO/Driver.cpp
  lld/MachO/Driver.h
  lld/MachO/DriverUtils.cpp
  lld/MachO/Options.td
  lld/test/MachO/Inputs/DependencyDump.py
  lld/test/MachO/dependency-info.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98559.332708.patch
Type: text/x-patch
Size: 8435 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210323/d484a4b8/attachment.bin>


More information about the llvm-commits mailing list