efriedma-quic wrote:

I mean, looking at the code, as far as I can tell, we always call GetAddrOfVTT 
immediately before EmitVTTDefinition, so setGVProperties() has already run, so 
running it again shouldn't do anything... unless somehow EmitVTTDefinition is 
overwriting the visibility, or setGVProperties() behaves differently for 
declarations vs. definitions.  If something like that is happening, I'd like to 
explicitly document it.

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

Reply via email to