Davide Italiano via llvm-dev
2018-May-04 21:53 UTC
[llvm-dev] llvm-mc-assemble-fuzzer broken
While playing with sanitizer in a downstream project, I found out this.
/Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/tools/llvm-mc-assemble-fuzzer/llvm-mc-assemble-fuzzer.cpp:207:32:
error: reference to type 'std::unique_ptr<MCCodeEmitter>' could
not
bind to an
lvalue of type 'llvm::MCCodeEmitter *'
UseDwarfDirectory, IP, CE, MAB, ShowInst));
^~
/Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/include/llvm/Support/TargetRegistry.h:514:66:
note: passing argument to parameter 'CE' here
std::unique_ptr<MCCodeEmitter>
&&CE,
^
/Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/tools/llvm-mc-assemble-fuzzer/llvm-mc-assemble-fuzzer.cpp:232:55:
error: too many arguments to function call, expected 3, have 4
MCOptions);
^~~~~~~~~
/Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/include/llvm/Support/TargetRegistry.h:401:3:
note: 'createMCAsmBackend' declared here
MCAsmBackend *createMCAsmBackend(const MCSubtargetInfo &STI,
^
2 errors generated.
Apparently, this has been broken for several months. I think I can fix
this particular instance, but I was wondering whether anybody cares
about this tool?
--
Davide
"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare
George Karpenkov via llvm-dev
2018-May-05 00:10 UTC
[llvm-dev] llvm-mc-assemble-fuzzer broken
It worked in August. Last time I’ve asked (again, in August) someone did seem to care, but it is inevitable it would bitrot if it’s not built in any of the bots. George> On May 4, 2018, at 2:53 PM, Davide Italiano via llvm-dev <llvm-dev at lists.llvm.org> wrote: > > While playing with sanitizer in a downstream project, I found out this. > > /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/tools/llvm-mc-assemble-fuzzer/llvm-mc-assemble-fuzzer.cpp:207:32: > error: reference to type 'std::unique_ptr<MCCodeEmitter>' could not > bind to an > lvalue of type 'llvm::MCCodeEmitter *' > UseDwarfDirectory, IP, CE, MAB, ShowInst)); > ^~ > /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/include/llvm/Support/TargetRegistry.h:514:66: > note: passing argument to parameter 'CE' here > std::unique_ptr<MCCodeEmitter> &&CE, > ^ > /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/tools/llvm-mc-assemble-fuzzer/llvm-mc-assemble-fuzzer.cpp:232:55: > error: too many arguments to function call, expected 3, have 4 > MCOptions); > ^~~~~~~~~ > /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/include/llvm/Support/TargetRegistry.h:401:3: > note: 'createMCAsmBackend' declared here > MCAsmBackend *createMCAsmBackend(const MCSubtargetInfo &STI, > ^ > 2 errors generated. > > Apparently, this has been broken for several months. I think I can fix > this particular instance, but I was wondering whether anybody cares > about this tool? > > -- > Davide > > "There are no solved problems; there are only problems that are more > or less solved" -- Henri Poincare > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
Davide Italiano via llvm-dev
2018-May-05 00:17 UTC
[llvm-dev] llvm-mc-assemble-fuzzer broken
Thank you. I went ahead with a speculative fix in r331568. I'm not familiar _at all_ with the tool, so, although the fix was straightforward, another pair of eyes from somebody familiar with the tool would be appreciated. On Fri, May 4, 2018 at 5:10 PM, George Karpenkov <ekarpenkov at apple.com> wrote:> It worked in August. > Last time I’ve asked (again, in August) someone did seem to care, > but it is inevitable it would bitrot if it’s not built in any of the bots. > > George > >> On May 4, 2018, at 2:53 PM, Davide Italiano via llvm-dev <llvm-dev at lists.llvm.org> wrote: >> >> While playing with sanitizer in a downstream project, I found out this. >> >> /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/tools/llvm-mc-assemble-fuzzer/llvm-mc-assemble-fuzzer.cpp:207:32: >> error: reference to type 'std::unique_ptr<MCCodeEmitter>' could not >> bind to an >> lvalue of type 'llvm::MCCodeEmitter *' >> UseDwarfDirectory, IP, CE, MAB, ShowInst)); >> ^~ >> /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/include/llvm/Support/TargetRegistry.h:514:66: >> note: passing argument to parameter 'CE' here >> std::unique_ptr<MCCodeEmitter> &&CE, >> ^ >> /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/tools/llvm-mc-assemble-fuzzer/llvm-mc-assemble-fuzzer.cpp:232:55: >> error: too many arguments to function call, expected 3, have 4 >> MCOptions); >> ^~~~~~~~~ >> /Users/davide/work/llvm-monorepo/llvm-project-20170507/llvm/include/llvm/Support/TargetRegistry.h:401:3: >> note: 'createMCAsmBackend' declared here >> MCAsmBackend *createMCAsmBackend(const MCSubtargetInfo &STI, >> ^ >> 2 errors generated. >> >> Apparently, this has been broken for several months. I think I can fix >> this particular instance, but I was wondering whether anybody cares >> about this tool? >> >> -- >> Davide >> >> "There are no solved problems; there are only problems that are more >> or less solved" -- Henri Poincare >> _______________________________________________ >> LLVM Developers mailing list >> llvm-dev at lists.llvm.org >> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >-- Davide "There are no solved problems; there are only problems that are more or less solved" -- Henri Poincare