This patch adds a subset of PowerPC vector load intrinsics - vec_ld, vec_lde, vec_ldl, vec_lxvp, vsx_lxvp, vec_xld2 and vec_xlw4.
Details
Diff Detail
Event Timeline
Changes:
- put the vec_lxvp test into a separate file
- change assert to llvm_unreachable
- fix a typo in ppc-vec-load.f90
flang/test/Lower/PowerPC/ppc-vec-load-pwr10.f90 | ||
---|---|---|
3 | For dumping the LLVM IR, we used ! RUN: %flang_fc1 -triple powerpc64le-unknown-unknown -target-cpu pwr10 -emit-llvm %s -o - | FileCheck --check-prefixes="CHECK" %s |
flang/test/Lower/PowerPC/ppc-vec-load-pwr10.f90 | ||
---|---|---|
3 | Is REQUIRES: target=powerpc{{.*}} sufficient? Do you want to exclude all non-powerpc64le targets for this test? |
flang/test/Lower/PowerPC/ppc-vec-load-pwr10.f90 | ||
---|---|---|
3 | To be more specific, this is what is in the other Power10 test file ! RUN: %flang_fc1 -triple powerpc64le-unknown-unknown -target-cpu pwr10 -emit-llvm %s -o - | FileCheck --check-prefixes="CHECK" %s ! REQUIRES: target=powerpc{{.*}} Since it is testing the LLVM IR, I think we use use %$flang_fc1 instead of %flang. Because -mcpu is not a -fc1 option, we used -target-cpu power10, so we no longer need -S . |
flang/test/Lower/PowerPC/ppc-vec-load-pwr10.f90 | ||
---|---|---|
3 | Thanks for the clarification. I will make the change accordingly. |
Should this be llvm_unreachable?