https://bugs.llvm.org/show_bug.cgi?id=42492
Chen Zheng (陈 正) <chenzheng1...@hotmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|NEW |RESOLVED
--- Comment #6 from Chen Zheng (陈 正) <chenzheng1...@hotmail.com> ---
https://reviews.llvm.org/D64197 introduces dependancy from Analysis ->
TransformUtils. The left hardeware loop checking code in HardwareLoops pass is
based on existing of loop preheader and if there is no preheader, HardwareLoops
pass inserts one. Inserting a preheader is ok in HardwareLoops pass, but it is
nok in TargetTransformInfo.cpp. TargetTransformInfo.cpp is in Analyses library,
currently it does not depend on TransformUtils library, so we meet symbol
undefined issue if we build with -DBUILD_SHARED_LIBS=ON.
Since the build crash should be fixed by https://reviews.llvm.org/rL365104 and
it is unavoidable that isHardwareLoopCandidate() returns different value in
CanSaveCmp() hook and HardwareLoops pass, I will abandon patch for item 1.
--
You are receiving this mail because:
You are on the CC list for the bug._______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs
