aaron.ballman added a comment.

I had a discussion about the license file on IRC (but not with @tstellar) and 
the thinking there was: add the license file. It seems to either be required or 
would be harmless to add. So if we don't hear something different from 
@tstellar on this, I think you should add the contents of 
https://www.unicode.org/license.txt to the top of the generated content for 
`UnicodeNameToCodepointGenerated.cpp` so that the license information stays 
with the derived data to which it applies.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123064

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

Reply via email to