tejohnson added a comment.

Also, just a nit, because TLI is sometimes used to refer to the 
TargetLibraryInfo and occasionally to the TargetLibraryInfoImpl, and the latter 
is frequently referred to as TLII, could you change the description to say TLII 
or TLI Impl? Since this doesn't affect TargetLibraryInfo/TLI object copies.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77952



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

Reply via email to