jsji added inline comments.

================
Comment at: clang/test/Driver/ppc-float-abi-warning.cpp:2
+// REQUIRES: powerpc-registered-target
+// RUN: %clang -### -x c++ -target powerpc64le-unknown-linux-gnu %s 
-mabi=ieeelongdouble
+
----------------
Can we add RUN line to test when we are using libc++?


================
Comment at: clang/test/Driver/ppc-float-abi-warning.cpp:4
+
+// expected-warning{{float ABI 'ieeelongdouble' is not supported by current 
C++ stdlib}}
+
----------------
So this assume that all our buildbots are using old glibc? What if we have 
mixed envs, eg: some with newer glibc?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112906

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

Reply via email to