https://github.com/tblah approved this pull request.

Flang changes LGTM.

nit: I am no clang expert but I wonder if you could use MarshallingInfoFlag in 
Options.td to avoid having to set CGOpts.FuseLoops manually for clang (this is 
not supported for flang unfortunately). If people more familiar with clang are 
happy with it how it is then that is fine by me.

Please wait for somebody else to review this from a clang/ and llvm/ 
perspective.

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

Reply via email to