mati865 added a comment.

> If we make the flag imply linker options, then it becomes much clearer to 
> spot the error, if you enabled this but the code base still would need 
> autoimports somewhere. (This has happened - see 
> https://code.videolan.org/videolan/dav1d/-/merge_requests/1303#note_301859, 
> https://code.videolan.org/videolan/dav1d/-/merge_requests/1349 and 
> https://code.videolan.org/videolan/dav1d/-/merge_requests/1361.) If we make 
> the flag only affect code generation, it becomes a more clear match for 
> projects using -mcmodel=small with GCC today.

LLVM already differs quite a bit from GNU in more "visible" parts like TLS so 
I'm don't see compelling reason to degrade user's experience just to be more 
compatible.
`-fno-autoimport` sounds fine for me.


Repository:
  rC Clang

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

https://reviews.llvm.org/D61670

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D61670: [RFC] [Min... Mateusz Mikuła via Phabricator via cfe-commits

Reply via email to