eddyz87 added a comment.

Slightly above my edit there is a similar logic for NVPTX target:

  static void RenderSSPOptions(const Driver &D, const ToolChain &TC,
                               const ArgList &Args, ArgStringList &CmdArgs,
                               bool KernelOrKext) {
    const llvm::Triple &EffectiveTriple = TC.getEffectiveTriple();
  
    // NVPTX doesn't support stack protectors; from the compiler's perspective, 
it
    // doesn't even have a stack!
    if (EffectiveTriple.isNVPTX())
      return;
    ...

I don't like it because it produces rather vague error message:

  warning: argument unused during compilation: '-fstack-protector' 
[-Wunused-command-line-argument]

However I'm hesitant to merge the NVPTX check with BPF check to avoid any 
potential changes in the NVPTX target behavior (e.g. different warning).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142046

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to