This is not a code testing center.
You should know whether your code is correct and does what you want - or not. Did you ever run it? If so, you know the answer.
If you code is not correct, and you can't figure out why, then you can post what your code shall do, what it does instead, what you tried and what you observed - and ask whether someone has an idea why it isn't working. But don't post code you never tested yourself, and ask us to grade it.
However, I doubt it will work: you activate interrupts for the timer, but never enable interrupts globally. You should add a bis_SR_register(GIE); before entering your endless while.