Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gccgo: treats nil as type #11567

Closed
dvyukov opened this issue Jul 3, 2015 · 2 comments
Closed

gccgo: treats nil as type #11567

dvyukov opened this issue Jul 3, 2015 · 2 comments
Milestone

Comments

@dvyukov
Copy link
Member

dvyukov commented Jul 3, 2015

gccgo successfully compiles the following program:

package n
func f(a nil)

It must not.
gcc version 6.0.0 2015070 (experimental) (GCC)

@dvyukov
Copy link
Member Author

dvyukov commented Jul 3, 2015

@paranoiacblack

@gopherbot
Copy link

CL https://golang.org/cl/12662 mentions this issue.

vries pushed a commit to vries/gcc that referenced this issue Aug 1, 2015
    
    Function declarations don't create a block where the variables listed
    in the parameter list are declared.  Because there are no variables
    declared, the types of the parameter variables is unchecked, allowing
    for invalid values to be used as the type.  This patch adds a special
    case to the check_types pass for function declarations.
    
    Fixes golang/go#11567.
    
    Reviewed-on: https://go-review.googlesource.com/12662


git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@226456 138bc75d-0d04-0410-961f-82ee72b054a4
@golang golang locked and limited conversation to collaborators Aug 5, 2016
asiekierka pushed a commit to WonderfulToolchain/gcc-ia16 that referenced this issue May 16, 2022
    
    Function declarations don't create a block where the variables listed
    in the parameter list are declared.  Because there are no variables
    declared, the types of the parameter variables is unchecked, allowing
    for invalid values to be used as the type.  This patch adds a special
    case to the check_types pass for function declarations.
    
    Fixes golang/go#11567.
    
    Reviewed-on: https://go-review.googlesource.com/12662

From-SVN: r226456
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants