klimek added a comment.

In https://reviews.llvm.org/D39900#921885, @krasimir wrote:

> Maybe we should further refactor `getRawStringStyle` into 
> `llvm::Optional<std::pair<style, delimiter>> getRawStringStyleAndDelimiter` 
> and that would nicely take care of the duplicated effort?


I thought about that, too, but I'm still convinced the right solution if we 
really care about the runtime is to cache the result in FormatToken itself.


https://reviews.llvm.org/D39900



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

Reply via email to