[PATCH] D64819: [UpdateTestChecks][Bug41532] Add handle of basic block names.

Whitney via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 23 04:12:49 PDT 2019


Whitney added a comment.

@nikic Thanks for the suggestion. I personally ran into the issue that reassociate-after-unroll.ll failed due to the different in basic block name, after modifying the O2 <https://reviews.llvm.org/owners/package/2/> pass pipeline. And modifying the script, was the suggested way to solve the failure. I am willing to put this change behind a flag if that's preferred, but it maybe hard for user to know when to use the flag, or even know the existence of it.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64819





More information about the llvm-commits mailing list