extremely tedious and uninventive but you might be english or danish so it might just suit you. Who knows apart from yourself, so if you like it then call it that.
However if you really want my opinion… it sucks. Illidan was a DH and before that a mage, not a warlock. Also while I am not the most creative person when it comes to names, buuuuut Arthas and Illidan have become turbo cringe names because every DK/DH is using variations of them.
It works if you join the Scryers. They were part of Illidan’s forces until they swapped sides.
So from a lore point of view, you were a follower of Illidan and your name reflects that. You kept this name when you switched to the Naaru because [insert reason here].
Name is a name. You make the name and the reputation.
Ive seen very honorable xxxCoKMuncherxxx named players and ive seen the most toxic players with name NobleKnight.
If you like it just use it. Maybe your Sin’dorei parents named your character in honor of Illidan or it’s just a coincidence or… who cares it’s just a name
Personally I wouldn’t pick such a name but that’s a good thing otherwise we’d all walk around with the same character names right?
Not much of a sense lorewise, but if you like the name, go ahead. That’s what I did when remaking this character for Classic on an RP server and no one bats an eye.