https://github.com/tarunprabhu commented:

Do we really want to maintain an equivalence with clang by keeping this in 
`CodeGenOptions` when it really ought to be a `TargetOption`? It looks like 
almost all of the handling of this option takes place in `flang`, so we 
probably shouldn't be bound to idiosyncracies in `clang` if we can help it.

It might make the option declaration messier unless conditional marshalling is 
possible (I don't know if it is or not). 

I don't have a very strong opinion on this. If others are ok with leaving this 
in `CodeGenOptions`, I'm happy to go along with it. What do you think?

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

Reply via email to