crobertsbmw

your 'e' line looks like was intended to be p2.2, which translates to 0x04. but when u set it low, u are actually setting p2.6 low.
 
looks like u can try and replace all 0x40 w/ 0x04, or better yet BIT2.

void delaySec(unsigned n) { do __delay_cycles(1000000); while(--n); }
 
Your code does not work because i, j, & k are not declared volatile, so the compiler removes the empty for() loops.

Try a clean and rebuild.
Also, do something in the loop, like an x=x+1 or nop;
 
I think you compiler is being clever and optimizing it out.

SCOTTY! MORE POWER!
 
To my eyes, it looks like you need more power than the function generator is going to provide.
 
In fact, I bet the function generator is going to output 1Vpp @ 10mA maximum.
 
A first thought would be to study the MOC3063TVM datasheet to see at what point it triggers. That is, the working voltage and load current to make it do something useful.
 
Also, may I suggest that you test the triac using 24VAC from a standard door bell transformer instead of the function generator? The bell transformer will lower 120VAC to 24VAC and make it less deadly to you but still provide enough current to do something.
 
You could also put an incandescent light bulb in the load side to give you a visual indicator of your circuit function. Either that or a DVM or 'scope.
 
On the input side, you could rig up a simple switch that applies the proper trigger voltage to the anode through a load resistor. You should tie the cathode to ground in that case.
 
What do you think?