[PATCH] D98856: Always emit error for wrong interfaces to scalable vectors, unless cmdline flag is passed.

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 7 10:34:29 PDT 2021


MaskRay added inline comments.


================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:5066
+    CmdArgs.push_back("-mllvm");
+    CmdArgs.push_back("-treat-scalable-fixed-error-as-warning");
+  }
----------------
sdesmalen wrote:
> MaskRay wrote:
> > Does this introduce an option in the CC1 command line for most cases? We should shorten the CC1 command line for common cases...
> The option is purely temporary and will be removed in the future. I guess it depends on the meaning of the word 'common' in this context, but it enables experimental use of Clang for scalable vectors, which is common if the targets support them. We'd like to remove this flag sooner rather than later, although there is work involved in LLVM before we can remove it.
> 
> Are you specifically concerned with the length (as in: number of characters) of the cc1 command?
> Are you specifically concerned with the length (as in: number of characters) of the cc1 command?

Yes.

I looked at a random SVE unrelated `clang -###` and found this option... Now I tried again and can't find the option now. Reducing CC1 length without making internal implementation difficult is generally good.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98856



More information about the cfe-commits mailing list