[PATCH] D53450: [Pipeliner] Ignore Artificial dependences while computing recurrences.

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 19 18:50:38 PDT 2021


jdoerfert reopened this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.
Herald added a project: LLVM.

The test is pretty much full of UB and run with O3 <https://reviews.llvm.org/owners/package/3/>, neither is good by itself and together that results in changes anywhere to break this test. D106308 <https://reviews.llvm.org/D106308> does cause the entire code to be deleted, and that is not the fault of D106308 <https://reviews.llvm.org/D106308>. I will XFAIL this test and you can repair it as you need. FWIW, I'd strongly suggest to *not* run O3 <https://reviews.llvm.org/owners/package/3/> at all as part of such tests.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D53450



More information about the llvm-commits mailing list