[PATCH] D63638: [clang][NewPM] Add new pass manager RUN lines to avx512f-builtins.c

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 20 22:53:55 PDT 2019


craig.topper created this revision.
craig.topper added reviewers: leonardchan, chandlerc.
craig.topper edited the summary of this revision.

This is split from D63174 <https://reviews.llvm.org/D63174> to see what it takes to get both pass managers to do the same thing

I've disable -O0 optnone and ran instsimplify on both outputs. This seems to get us converged with some test updates. The only thing I don't like is the tests of true and false comparison predicates with masking. The only IR we ended up with is just loads and stores. The cmp and AND instruction are folded out by instsimplify.

I'll do more tests from D63174 <https://reviews.llvm.org/D63174> if we think this is a decent direction


https://reviews.llvm.org/D63638

Files:
  clang/test/CodeGen/avx512f-builtins.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63638.205942.patch
Type: text/x-patch
Size: 16623 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190621/6470f81c/attachment.bin>


More information about the llvm-commits mailing list