[PATCH] D21165: Disable MSan-hostile loop unswitching.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 8 18:47:50 PDT 2016


chandlerc added a comment.

Thanks Sanjoy for thinking about this.

Provided that my reasoning holds (this is only a problem for MSan because it is essentially treating branch-on-uninit as a side-effect, I have one more high level question: is that even remotely reasonable?

Should the MSan report be suppressed until the side-effect bearing instruction is executed? I think that would cause MSan to not have a false positive here. I'd love to hear the thoughts of the MSan people here.

-Chandler


Repository:
  rL LLVM

http://reviews.llvm.org/D21165





More information about the llvm-commits mailing list