[PATCH] D27838: Extract a TBAAVerifier out of the verifier (NFC)

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 15 19:03:03 PST 2016


mehdi_amini added a comment.

In https://reviews.llvm.org/D27838#624648, @sanjoy wrote:

> This is just code motion, right?  If so LGTM.


Mostly: it is moving the code outside of the anonymous namespace and grouping it in its own class, but yes the body of the function is unchanged, except that `visitTBAAMetadata` also returns a boolean now (ignored by the verifier).

I didn't upload the full patch before (didn't squash the early refactoring), you may want to look again.


https://reviews.llvm.org/D27838





More information about the llvm-commits mailing list