[PATCH] D32275: [globalisel][tablegen] Add several GINodeEquiv's for operators that do not require additional support.

Daniel Sanders via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 25 02:46:37 PDT 2017


dsanders added a comment.

I decided to have a look at code coverage differences before committing this. This patch made no difference to the code coverage. On closer inspection it turns out beginFunction() isn't being executed so no rules with rule predicates can match. I'll take a look at this, maybe something from the rule-predicates patch didn't make it into the final commit.

A few other notable things in the coverage information:

- There's no coverage for 64-bit shifts.
- There's no coverage for 8 or 16-bit FPU stores. That's not particularly surprising but I am surprised that there's an implementation for those sizes.
- There's no coverage of 128-bit FPU copies
- Most integer and FPU comparison predicates aren't covered.
- There's no coverage for LOAD_STACK_GUARD
- There's no coverage for G_TRUNC in the FPU register bank.


https://reviews.llvm.org/D32275





More information about the llvm-commits mailing list