aeubanks added a comment.

In D119011#3297305 <https://reviews.llvm.org/D119011#3297305>, @Anastasia wrote:

> Just to understand the intent - is this a performance optimization or 
> functionality fix?

This is a functionality fix.

> Also is there any way to test this?

I came across this while working on another patch. This should be NFC with ToT 
clang, but was required my other patch. Since this part was easily split apart 
I wanted to do that.
Separately I think this makes sense in general.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119011/new/

https://reviews.llvm.org/D119011

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

Reply via email to