[PATCH] D57444: [DebugInfo] LLVM ERROR: Broken function found, while removing Debug Intrinsics

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 8 16:53:44 PST 2019


aprantl added a comment.

In D57444#1390579 <https://reviews.llvm.org/D57444#1390579>, @CarlosAlbertoEnciso wrote:

> In D57444#1390350 <https://reviews.llvm.org/D57444#1390350>, @dstenb wrote:
>
> > Sorry for not noticing this before, but we noticed now that the test case is placed under `test/CodeGen/X86/`, although it is a simplifycfg test. Should it ideally be placed under `test/Transforms/SimplifyCFG/`?
>
>
> As this review is closed and committed, I will create another revision just to move the test to the 'Transforms' directory. Thanks


Don't bother with a review, just do it.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57444





More information about the llvm-commits mailing list