bcain added inline comments.

================
Comment at: clang/lib/Driver/ToolChains/Linux.cpp:923
+  return ((getTriple().isAndroid() && !getTriple().isAndroidVersionLT(16)) ||
+          getTriple().isMusl() || getSanitizerArgs().requiresPIE()) &&
+          getTriple().getArch() != llvm::Triple::hexagon;
----------------
But if 
```
getSanitizerArgs().requiresPIE() && getTriple().getArch() == 
llvm::Triple::hexagon
```

.... then the `!= hexagon` takes precedence?  Seems like "sanitizer requires 
PIE" should, right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74260



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

Reply via email to