https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
Yury Gribov changed:
What|Removed |Added
Target Milestone|--- |4.9.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
Yury Gribov changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
CC|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #9 from ygribov at gcc dot gnu.org ---
Author: ygribov
Date: Wed Feb 5 05:22:29 2014
New Revision: 207497
URL: http://gcc.gnu.org/viewcvs?rev=207497root=gccview=rev
Log:
PR sanitizer/59600
gcc/
* cif-code.def
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #10 from Yury Gribov y.gribov at samsung dot com ---
Paul, does this work for you?
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #11 from eggert at gnu dot org ---
It's not easy for me to test the fix, sorry, I don't normally build GCC. If it
works on the test case my guess is that it'll work for Emacs, though.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #6 from Andrew Pinski pinskia at gcc dot gnu.org ---
Comment on attachment 31516
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=31516
New patch based on Andrew's review
No this wrong in two ways. Move the check to before the check
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #7 from Yury Gribov y.gribov at samsung dot com ---
(In reply to Andrew Pinski from comment #6)
Move the check to before the check of the target
attribute table.
My bad.
And second you should compare the current function
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #8 from Yury Gribov y.gribov at samsung dot com ---
Created attachment 31522
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=31522action=edit
Patch which inlines based on caller/callee attribute match
(In reply to Yury Gribov from
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
Yury Gribov y.gribov at samsung dot com changed:
What|Removed |Added
CC||y.gribov at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #2 from Kostya Serebryany kcc at gcc dot gnu.org ---
We had this problem in clang before
http://llvm.org/viewvc/llvm-project?view=revisionrevision=187967
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #3 from Andrew Pinski pinskia at gcc dot gnu.org ---
(In reply to Yury Gribov from comment #1)
Created attachment 31515 [details]
Draft patch
Why can't you just set DECL_UNINLINABLE on the function decl in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
--- Comment #4 from Andrew Pinski pinskia at gcc dot gnu.org ---
(In reply to Andrew Pinski from comment #3)
(In reply to Yury Gribov from comment #1)
Created attachment 31515 [details]
Draft patch
Why can't you just set DECL_UNINLINABLE
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59600
Yury Gribov y.gribov at samsung dot com changed:
What|Removed |Added
Attachment #31515|0 |1
is
13 matches
Mail list logo