[PATCH] D62225: [clang][NewPM] Fixing -O0 tests that are broken under new PM

Leonard Chan via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 21 16:46:58 PDT 2019


leonardchan created this revision.
leonardchan added reviewers: chandlerc, echristo, phosek, serge-sans-paille.
leonardchan added a project: clang.
Herald added subscribers: kristof.beyls, javed.absar.

This is a patch that should go on top of D58375 <https://reviews.llvm.org/D58375> which addresses/fixes tests that fail under new PM with -O0.

This is still ongoing and will ask for reviews once all of them are fixed. These are currently the latest ones I have updated so far.

Current reasons for fixes:

- Some aarch64 tests specified attribute numbers explicitly where as new pm uses the same attributes but different numbers.
- `CodeGen/aarch64-neon-perm.c` failed because new PM would insert lifetime start/end intrinsics even under -O0.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D62225

Files:
  clang/lib/CodeGen/BackendUtil.cpp
  clang/test/CodeGen/aarch64-neon-across.c
  clang/test/CodeGen/aarch64-neon-fcvt-intrinsics.c
  clang/test/CodeGen/aarch64-neon-fma.c
  clang/test/CodeGen/aarch64-neon-perm.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62225.200606.patch
Type: text/x-patch
Size: 28452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190521/283fdcda/attachment-0001.bin>


More information about the cfe-commits mailing list