Maetveis wrote:

@tarunprabhu Continuing [our discussion on 
#108868](https://github.com/llvm/llvm-project/pull/108868/files/612b93eda5695306293bb8ca6e27a810b06adf3d#r1867949328)
 here. These are what I found, by searching for commits that added both 
`FlangOption` and `CLOption` to Options.td and were still reachable in the 
blame of the file. It is possible that this is not all of them.
The default can get quite confusing to track because IIUC the `OptionGroup`'s 
`Visibility` gets unioned with the visibility defined on the record.
 

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

Reply via email to