This makes it a bit clearer why we're saving the token (so no need for the later comment).
I'm just not 100% sure about the lifetime of the token name.
Differential D147615
[clang][Sema][NFC] Save token name instead of the full token tbaeder on Apr 5 2023, 7:25 AM. Authored by
Details This makes it a bit clearer why we're saving the token (so no need for the later comment). I'm just not 100% sure about the lifetime of the token name.
Diff Detail
Event Timeline
Comment Actions LGTM! There shouldn't be any lifetime issues from this, as @shafik pointed out, the pointer originates in a static array defined in TokenKinds.cpp. |
It looks like this comes from a static array so it should be fine but let's see what Aaron says.