[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