tejohnson added inline comments.

================
Comment at: clang/test/Driver/hip-options.hip:72
+// HIPTHINLTO-NOT: error: invalid argument '-fwhole-program-vtables' only 
allowed with '-flto'
+// HIPTHINLTO-NOT: clang{{.*}} "-triple" "x86_64-unknown-linux-gnu" {{.*}} 
"-flto-unit"
+// HIPTHINLTO: clang{{.*}} "-triple" "amdgcn-amd-amdhsa" {{.*}} "-flto=thin" 
"-flto-unit" {{.*}} "-fwhole-program-vtables"
----------------
These are essentially the same checks as before, but I renamed this tag to 
HIPTHINLTO, added a check to ensure we don't get the error with 
-fwhole-program-vtables, ensure -fwhole-program-vtables passed through for the 
offload cc1 command, and added a duplicate of the HIPTHINLTO-NOT check (since 
in my local build the non-offload cc1 is emitted after the offload cc1, not 
sure if there is some variation).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103579

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

Reply via email to