gAlfonso-bit added a comment.

In D110068#3013636 <https://reviews.llvm.org/D110068#3013636>, @dexonsmith 
wrote:

> I'm a bit hesitant to trust lack of tests as proof that old FIXMEs can be 
> resolved safely. Can you add some background information about why this was 
> originally a specifier, and why it's safe to fix it now? (If you don't know, 
> I suggest looking through git-blame to figure it out (unless @ahatanak 
> already knows?).)

I did and it was originally put there when the function was made. Not because 
of a bug.


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

https://reviews.llvm.org/D110068

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

Reply via email to