[PATCH] D92842: [SelectionDAG] Add Target-Independent Compiler Barrier

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 16 02:13:44 PDT 2021


lenary added a comment.

In D92842#2821463 <https://reviews.llvm.org/D92842#2821463>, @nigelp-xmos wrote:

> llvm/include/llvm/Target/TargetSelectionDAG.td defines SDTMemBarrier. This seems to have been introduced in 9b254eed32028 for ISD::MEMBARRIER, which was removed some time ago. I can't find a use of it and I can build without it. Could SDTMemBarrier be removed and would it belong in this patch?

Yes, that makes sense to me, I think I removed some target-specific `TypeProfile`s which were now unused, so it makes sense to remove this too.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92842/new/

https://reviews.llvm.org/D92842



More information about the llvm-commits mailing list