https://github.com/hokein commented:

The change looks good. We should be mindful of the potential performance 
regression due to this (1% increase in preamble size, as noted 
https://reviews.llvm.org/D124688#3483895).  

I’ll leave the final approval to @kadircet.  


https://github.com/llvm/llvm-project/pull/127359
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to