Antes de ponerte tus errores de sintaxis, dejame darte un consejo, el utilizar variables globales es una tentadora forma de trabajar que suene muy eficiente, pero la realidad es que no es nada eficiente, tal ves ahorita no lo veas, pero al tener que realizar un proyecto mas extenso te daras cuenta que esto no es eficiente. asi que mejor cambia eso, tienes que pasas valores por parametro, asi podria quedar tu funcion
Código C++:
Ver originalint suma(int a, int b)
{
int c;
c=a+b;
return c;//es el valor q
}
//desde el main le envias los valores de a y b asi
int c;
cin>>a>>b;
c=suma(a,b);
errores:
1. al leer la opcion lo intentas meter en una variable llamada "opc" la cual no esta declarada.
2. el default tambien lleba los dos puntos como los case.
3. cuando intentas leer los valores de las variables, si quieres leerelas las dos al mismo tiempo(que no es aconsejable) tienes que hacerlo asi cin>>n1>>n2;