rsmith added a comment.

Looks OK to me, pending Bruno's confirmation that performance is acceptable.


================
Comment at: lib/Lex/PPDirectives.cpp:102

----------------
Can you use `llvm::StringSwitch` for this? I'd be interested to see how the 
performance compares.


http://reviews.llvm.org/D19843



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to