public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug ipa/61555] [4.9/4.10 Regression] LLVM build failure [not found] <bug-61555-4@http.gcc.gnu.org/bugzilla/> @ 2014-06-18 20:43 ` pinskia at gcc dot gnu.org 2014-06-18 20:46 ` trippels at gcc dot gnu.org ` (2 subsequent siblings) 3 siblings, 0 replies; 4+ messages in thread From: pinskia at gcc dot gnu.org @ 2014-06-18 20:43 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61555 Andrew Pinski <pinskia at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |INVALID --- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> --- Are you sure this is not a LLVM source issue? llvm::cl::parser<llvm::PassInfo const*>::getOption(unsigned int) const is called by llvm::cl::C<llvm::D>::getExtraOptionNames(int&). llvm::cl::C<llvm::D>::getExtraOptionNames(int&) is part of vtable for llvm::cl::C<llvm::D>. I don't see anything wrong here really as the type of Parser can found at compile time and it is D inherits from cl::parser<const PassInfo *>. So getExtraOptionNames calls Parser.getExtraOptionNames which in turns calls this->getOption (0) which is a virtual function call. Since we know the type of *this here, we call directly llvm::cl::parser<llvm::PassInfo const*>::getOption(unsigned int) const. So the devirtuatization seems correct and it is just a LLVM source problem. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug ipa/61555] [4.9/4.10 Regression] LLVM build failure [not found] <bug-61555-4@http.gcc.gnu.org/bugzilla/> 2014-06-18 20:43 ` [Bug ipa/61555] [4.9/4.10 Regression] LLVM build failure pinskia at gcc dot gnu.org @ 2014-06-18 20:46 ` trippels at gcc dot gnu.org 2014-06-27 15:47 ` mustrumr97 at gmail dot com 2014-06-30 15:12 ` rafael.espindola at gmail dot com 3 siblings, 0 replies; 4+ messages in thread From: trippels at gcc dot gnu.org @ 2014-06-18 20:46 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61555 --- Comment #2 from Markus Trippelsdorf <trippels at gcc dot gnu.org> --- (In reply to Andrew Pinski from comment #1) > Are you sure this is not a LLVM source issue? > > llvm::cl::parser<llvm::PassInfo const*>::getOption(unsigned int) const > is called by llvm::cl::C<llvm::D>::getExtraOptionNames(int&). > > llvm::cl::C<llvm::D>::getExtraOptionNames(int&) is part of vtable for > llvm::cl::C<llvm::D>. > > > I don't see anything wrong here really as the type of Parser can found at > compile time and it is D inherits from cl::parser<const PassInfo *>. > So getExtraOptionNames calls Parser.getExtraOptionNames which in turns calls > this->getOption (0) which is a virtual function call. Since we know the > type of *this here, we call directly llvm::cl::parser<llvm::PassInfo > const*>::getOption(unsigned int) const. > > So the devirtuatization seems correct and it is just a LLVM source problem. Thanks. I just wasn't 100% sure. LLVM bug: http://llvm.org/bugs/show_bug.cgi?id=20067 ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug ipa/61555] [4.9/4.10 Regression] LLVM build failure [not found] <bug-61555-4@http.gcc.gnu.org/bugzilla/> 2014-06-18 20:43 ` [Bug ipa/61555] [4.9/4.10 Regression] LLVM build failure pinskia at gcc dot gnu.org 2014-06-18 20:46 ` trippels at gcc dot gnu.org @ 2014-06-27 15:47 ` mustrumr97 at gmail dot com 2014-06-30 15:12 ` rafael.espindola at gmail dot com 3 siblings, 0 replies; 4+ messages in thread From: mustrumr97 at gmail dot com @ 2014-06-27 15:47 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61555 Hristo Venev <mustrumr97 at gmail dot com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mustrumr97 at gmail dot com --- Comment #3 from Hristo Venev <mustrumr97 at gmail dot com> --- In the LLVM codebase getOption() is defined but still not written in the object file. The reduced test case is wrong. Why can't I reopen this bug? ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug ipa/61555] [4.9/4.10 Regression] LLVM build failure [not found] <bug-61555-4@http.gcc.gnu.org/bugzilla/> ` (2 preceding siblings ...) 2014-06-27 15:47 ` mustrumr97 at gmail dot com @ 2014-06-30 15:12 ` rafael.espindola at gmail dot com 3 siblings, 0 replies; 4+ messages in thread From: rafael.espindola at gmail dot com @ 2014-06-30 15:12 UTC (permalink / raw) To: gcc-bugs https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61555 Rafael Avila de Espindola <rafael.espindola at gmail dot com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rafael.espindola at gmail dot com --- Comment #4 from Rafael Avila de Espindola <rafael.espindola at gmail dot com> --- A valid testcase reduction is at https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61659 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-06-30 15:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-61555-4@http.gcc.gnu.org/bugzilla/>
2014-06-18 20:43 ` [Bug ipa/61555] [4.9/4.10 Regression] LLVM build failure pinskia at gcc dot gnu.org
2014-06-18 20:46 ` trippels at gcc dot gnu.org
2014-06-27 15:47 ` mustrumr97 at gmail dot com
2014-06-30 15:12 ` rafael.espindola at gmail dot com
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).