[PATCH] D68825: [libTooling] Change Stencil equality to use `toString()`

Yitzhak Mandelbaum via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 10 12:37:15 PDT 2019


ymandel created this revision.
ymandel added a reviewer: gribozavr.
Herald added a project: clang.
ymandel edited the summary of this revision.

Removes the `isEqual` method from StencilPartInterface and modifies equality to
use the string representation returned by the `toString` method for comparison.

This means the `run` and `selection` stencils return true by default, and
clients should be cautious in relying on equality operator for comparison of
stencils containing parts generated by these functions.

It also means we no longer need the custom RTTI support, so it has been removed.

Patch by Harshal T. Lehri.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D68825

Files:
  clang/include/clang/Tooling/Transformer/Stencil.h
  clang/lib/Tooling/Transformer/Stencil.cpp
  clang/unittests/Tooling/StencilTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68825.224448.patch
Type: text/x-patch
Size: 5917 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191010/bffd5c1a/attachment.bin>


More information about the cfe-commits mailing list